Could there be a way to suppress warnings?

[redspacecat]

Currently when I build my project, I get 20+ warnings from things like unused procedure arguments, struct fields, structs, procedures, etc.
I have many things there for later or things that I don't care about. The warnings clutter up finding errors when they occur and are quite inconvenient.

Could there please be a way to suppress warnings in the output while building?
Possibly a flag when doing goboscript build
Example: goboscript build --suppress-warnings

[linear]

GS-290

[aspizu]

To suppress unused warnings, prefix the thing with an underscore.

Let me know if any of the warnings are invalid, I would want to add a way to suppress those.

[redspacecat]

To suppress unused warnings, prefix the thing with an underscore.

Let me know if any of the warnings are invalid, I would want to add a way to suppress those.

Sorry prefix what "thing"?

[redspacecat]

I went though my warnings and indeed many of them are incorrect.

Incorrect warnings 14/22

warning: unused struct InputRank
  --> stage.gs:10:8
   |
10 | struct InputRank {
   |        ---------
   |
warning: unused struct field dist (never read)
  --> stage.gs:11:5
   |
11 |     dist="",
   |     ----
   |
warning: unused struct field block (never read)
  --> stage.gs:12:5
   |
12 |     block="",
   |     -----
   |
warning: unused struct field input (never read)
  --> stage.gs:13:5
   |
13 |     input="" # input id in block
   |     -----
   |

Here I'm using InputRank in stage.gs, and even though the InputRank definition is also included elsewhere, it throws an error if I get rid of the definition here.

warning: unused struct TextLabel
 --> code\helpful_utils.gs:1:8
  |
1 | struct TextLabel {
  |        ---------
  |
warning: unused struct field text (never read)
 --> code\helpful_utils.gs:2:5
  |
2 |     text="",
  |     ----
  |
warning: unused struct field x (never read)
 --> code\helpful_utils.gs:3:5
  |
3 |     x=""
  |     -
  |
warning: unused struct Input
 --> code\helpful_utils.gs:6:8
  |
6 | struct Input {
  |        -----
  |
warning: unused struct field shape (never read)
 --> code\helpful_utils.gs:7:5
  |
7 |     shape="",
  |     -----
  |
warning: unused struct field x (never read)
 --> code\helpful_utils.gs:8:5
  |
8 |     x="",
  |     -
  |
warning: unused struct field width (never read)
 --> code\helpful_utils.gs:9:5
  |
9 |     width="",
  |     -----
  |
warning: unused struct field height (never read)
  --> code\helpful_utils.gs:10:5
   |
10 |     height="",
   |     ------
   |
warning: unused struct field text (never read)
  --> code\helpful_utils.gs:11:5
   |
11 |     text="",
   |     ----
   |
warning: unused struct field nested_block (never read)
  --> code\helpful_utils.gs:12:5
   |
12 |     nested_block="",
   |     ------------
   |

For all of these, the structs and all those fields are being used in one of the files that helpful_utils.gs is %included in, so this is incorrect.

For the legitimate ones, I figured out that you have to prefix the name of whatever it is with "_". This is inconvenient and I would prefer a way to just hide warnings completely.

[faretek1]

i think something more like #noqa or #type:ignore comments would be a better idea for case-by-case suppression in the more general case

[aspizu]

now warnings are printed before errors

[redspacecat]

now warnings are printed before errors

That definitely helps since it makes it easier to find the errors. Thanks!