search

bsermons / flycheck-elm

Flycheck support for the elm language
GNU General Public License v3.0
18 stars 8 forks source link

Fix #4 - Add option-var to set the output file #6

Closed trezona-lecomte closed 8 years ago

trezona-lecomte commented 8 years ago

This change adds an option-var for setting the file to be passed as the --output argument to elm-make whenever flycheck-elm is run.

The file defaults to index.html, the current default for elm-make.

bsermons commented 8 years ago

This doesn't exactly work on windows. I think the problem is elm-make wants the file to end in .html or .js so it spits out an error. I can test this by manually running: elm make main.elm --output=NUL Oddly, if I do elm make main.elm --output=NUL.js it seems to work.

Need to test on linux or OSX and see if elm make main.elm --output=/dev/null actually compiles or if it has similar behavior and then go from there.

bsermons commented 8 years ago

Merged with some minor modifications. elm-make has special logic for /dev/null so I just hard coded that. Thanks for you help on this.

trezona-lecomte commented 8 years ago

Thanks very much for getting this working @bsermons! I appreciate it.