search

mybuddymichael / linter-elm-make

Lint your Elm files in Atom with elm-make
MIT License
32 stars 12 forks source link

Report warnings doesn't work unless Lint on the Fly is chosen #79

Open cscalfani opened 8 years ago

cscalfani commented 8 years ago

This is occurring in version 0.16.0 (it occurred in previous versions too if that helps).

halohalospecial commented 8 years ago

Hi @cscalfani, thanks for the report! Unfortunately, I could not reproduce this :( The Report Warnings option is working regardless if Lint On The Fly is enabled or disabled. Can you kindly give me your linter-elm-make and linter settings (they're in ~/.atom/config.cson) so that I can investigate? Thanks!

cscalfani commented 8 years ago

Not sure this will be too helpful, but here it is:

linter:
    errorPanelHeight: 99
"linter-elm-make": {}
halohalospecial commented 8 years ago

Haha! Can you run Atom using atom -d and check the console log. It should show the arguments passed to elm-make (e.g. [linter-elm-make] Executing elm-make Main.elm --report=json --output=/dev/null --yes --warn)

halohalospecial commented 8 years ago

Thanks again @cscalfani for helping me debug this :-) I already submitted an issue to the elm-make repository: https://github.com/elm-lang/elm-make/issues/114