search

codewars / codewars-runner-cli

Old CodeRunner project. See https://github.com/codewars/runner instead.
GNU Affero General Public License v3.0
402 stars 141 forks source link

PureScript: disable warnings for unspecified imports #703

Closed Voileexperiments closed 5 years ago

Voileexperiments commented 5 years ago

Unspecified imports are warnings for some reasons, and completely clutters the runner output.

Also maybe it'd be good to disable warnings in general?

kazk commented 5 years ago

completely clutters the runner output.

Warnings in stderr should be collapsed when the tests pass. It's expanded on failure and that's intentional because warnings can contain useful information when something is wrong.

There's no way to turn off only unspecified imports warning as far as I know using the standard tools.

Voileexperiments commented 5 years ago

The problem is, warnings and errors are all printed to stderr, and whenever there's an compile error the error always goes after the warnings, which makes it very hard to read the error message.

kazk commented 5 years ago

Yeah, I agree. stderr currently doesn't support any formatting if I remember correctly. So we'll need to improve that first. I also need to know if it's possible to get those errors/warnings in JSON or something so that we can display them better.

kazk commented 5 years ago

Opened #705