search

mpizenberg / elm-test-rs

Fast and portable executable to run your Elm tests
BSD 3-Clause "New" or "Revised" License
81 stars 13 forks source link

Accept --output=/dev/null #102

Closed mpizenberg closed 3 years ago

mpizenberg commented 3 years ago

This is intended to make --output=/dev/null a valid argument, even though it is ignored. All other values for --output are forbidden and make elm-test-rs crash.

The goal is to ease editor integration, by behaving similar than elm make --output=/dev/null. I don't know if that's worth it, I'll let @lydell and @razzeee decide for this one.

lydell commented 3 years ago

This could potentially simplify the language server code a little bit, but isn’t needed.

But people who switch from node-test-runner might appreciate this. Emacs users especially, according to https://github.com/rtfeldman/node-test-runner/issues/508.

razzeee commented 3 years ago

I have no opinion on this.

mpizenberg commented 3 years ago

Alright thanks all. It's merged then.