search

reasonml / reason-cli

Globally installable Reason toolchain.
MIT License
291 stars 23 forks source link

CI option for refmt to check only #90

Closed Eric-Arellano closed 2 years ago

Eric-Arellano commented 5 years ago

Hello, thank you for making this awesome tool!

It looks like refmt has no way to only check, i.e. return an error code of 1 if any changes will be made but don't actually make those changes, correct? When ran without flags and when ran with --in-place, it returns a code of 0, unless there was a syntax issue.

The use case for this feature is CI. It doesn't make sense for my CI to actually try to rewrite the files. I instead want it to error if a rewrite is necessary.

--

I'm happy to work on a PR to fix this, but wanted to run it by you all first.

Eric-Arellano commented 5 years ago

Hm it looks like this is the wrong repository to post this? I'm having a tough time finding where refmt is actually implemented.