Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request to suppress warnings in --daemon context #988

Closed
RieksJ opened this issue Jul 9, 2019 · 1 comment
Closed

Feature request to suppress warnings in --daemon context #988

RieksJ opened this issue Jul 9, 2019 · 1 comment
Assignees
Labels
component: compiler optimization Not a bug, but something that could be made better. In many cases, this can require some rethinking.

Comments

@RieksJ
Copy link
Contributor

RieksJ commented Jul 9, 2019

When developing a script in VSCode, I use the Ampersand Checker (ampersand with --daemon switch) to locate the obvious mistakes I make - this is a real asset.

The --daemon switch also provides warnings. Some of these warnings (e.g. CRUD-related warnings that indicate a possible misunderstanding of the developer) are easily fixed. However, other warnings exist that may be helpful to beginners, but are a nuisance for expert developers.

Here is an example:

sbiNNnnn   :: SBIEntry * SBIDigits   [UNI] -- first digit pair of SBI code.
sbinnNNn   :: SBIEntry * SBIDigits   [UNI] -- second digit pair (if it exists).
sbinnnnN   :: SBIEntry * SBIDigits   [UNI] -- fifth digit (if it exists).
sbiNNxNNxN :: SBIEntry * SBICode [UNI,INJ] -- all digits, pairs are separated with dots.

where the relations all contain one part of a so-called SBICode. Note that the names of the relations precisely indicate which part of the code is intended, and therefore should remain as they are. However, this leads to the daemon continuously providing the following messages:

Unknown origin warning:
  Ampersand is case sensitive. you might have meant that the following are equal:
      Relation`sbiNNnnn` and `sbinnNNn`.
Unknown origin warning:
  Ampersand is case sensitive. you might have meant that the following are equal:
      Relation`sbiNNnnn` and `sbinnnnN`.
Unknown origin warning:
  Ampersand is case sensitive. you might have meant that the following are equal:
      Relation`sbinnNNn` and `sbinnnnN`.

This issue calls for a means that allows developers to (selectively) turn of warnings in daemon mode.

One possibility might be to introduce a switch --noWarnings that would simply not provide warnings. Note that this issue does not (necessarily) call for adaptation of the VSCode stuff. It just calls for something like this option.

@RieksJ RieksJ added optimization Not a bug, but something that could be made better. In many cases, this can require some rethinking. component: compiler labels Jul 9, 2019
@hanjoosten
Copy link
Member

The switch is called --no-warnigns.

hanjoosten added a commit that referenced this issue May 31, 2020
@RieksJ RieksJ closed this as completed Jun 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: compiler optimization Not a bug, but something that could be made better. In many cases, this can require some rethinking.
Projects
None yet
Development

No branches or pull requests

2 participants