Stack: Make stack warning about missing other-modules into an error (with flags)

Created on 9 Nov 2015  路  5Comments  路  Source: commercialhaskell/stack

If I run stack build it says:

Warning: modules not listed in shake.cabal for 'shake' component (add to other-modules):
    Development.Ninja.Env

How can I turn that into an error so I can test for it in my continuous integration? stack build --Werror? Add a separate stack check target?

awaiting pull request newcomer friendly enhancement

Most helpful comment

This looks like a fun nugget to chew on. I'll tackle this. :)

All 5 comments

One option would be to extend the meaning of --pedantic to also turn warnings like this into errors (only for things that are always resolvable).

I am happy to run with --Werror but my code is not --Wall clean, and I consider several of the Wall warnings to be actively harmful to code. Perhaps Stack should support both flags separately, as well as together with pedantic, then just Werror would have this effect.

I love this new flag idea, and also agree that not all Wall warnings should become errors with this flag

If anyone wants to work on a PR, that would be welcome.

This looks like a fun nugget to chew on. I'll tackle this. :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Toxaris picture Toxaris  路  4Comments

sjakobi picture sjakobi  路  4Comments

domenkozar picture domenkozar  路  3Comments

bitemyapp picture bitemyapp  路  3Comments

rrnewton picture rrnewton  路  4Comments