Support -warn-error argument

[aspeddro]

Add supoort for -warn-error argument. Use for CI only.

If there is already a configuration for warnings as error in rescript.json the argument will be added at the end

"warnings": {
  "error": "+27"
}

Example: Disable warning 27, run rescript -warn-error -27 generate -warn-error +27-27, the last one wins.

Related #6713

[cknitt]

Thanks for your work! Some remarks:

1. When passing an invalid an invalid argument like rescript -warn-error 32 instead of rescript -warn-error +32, I get the following error message:

Fatal error: exception Stdlib.Arg.Bad("Ill-formed list of warnings")

Maybe we could print a nicer error message in this case.

2. I tested this and it did not work for me in a subpackage in a monorepo setup.

[cknitt]

It is unclear here that a list of warning codes is needed as an argument (and what format this list needs to have).

[aspeddro]

The new error message

Argument description now is

-warn-error  Warning numbers and whether to turn it into error or no

Do you have an example of a monorepo?

[fhammerschmidt]

https://github.com/cca-io/rescript-mui

[cknitt]

@aspeddro Are you able to reproduce the problem with subpackages in the monorepo that @fhammerschmidt linked to or do you need a different example project?

[aspeddro]

I got an error installing local package with yarn

yarn add ~/Desktop/projects/rescript-compiler

Range Error: File size (2253956185) is greater than 2 GiB
    at new NodeError (node:internal/errors:405:5)
    at FSReqCallback.readFileAfterStat [as oncomplete] (node:fs:351:11)

I made a small example here

./node_modules/.bin/rescript build -with-deps -warn-error +110

Dependency pinned on shared
rescript: [1/3] src/Shared.ast
FAILED: src/Shared.ast

  Warning number 110 (configured as error) 
  /tmp/todo-monorepo-test/shared/src/Shared.res:1:21-25

  1 │ let add = (_, _) => %todo
  2 │ 

  Todo found.

  This code is not implemented yet and will crash at runtime. Make sure you implement this before runnin
g the code.

FAILED: cannot make progress due to previous errors.
Failure: /home/pedro/Desktop/projects/rescript-compiler/linux/ninja.exe 
Location: /tmp/todo-monorepo-test/node_modules/shared/lib/bs

[cknitt]

Retested with a monorepo, works fine now! 🎉 Thanks a lot!