deadfoxygrandpa / Elm.tmLanguage

Elm language syntax highlighting and tool integration for ST2/3
https://packagecontrol.io/packages/Elm%20Language%20Support
MIT License
122 stars 27 forks source link

Run elm-format on Save by default #68

Closed rtfeldman closed 8 years ago

rtfeldman commented 8 years ago

In my experience, if you have elm-format installed, the default use case is that you want it to run on Save - since that's the whole point!

I'd definitely like that to be the default behavior, to save me the trouble of configuring it - naturally, with the option to disable it (or, more likely, add a regexp filter specific to my development environment) if I don't want that default.

deadfoxygrandpa commented 8 years ago

I wanted to merge this into develop instead of into master, but I forgot about this and did the same change myself in commit 8294389e53f4a47d44773071a10ec3ae41987cbb

Thanks for the idea though

rtfeldman commented 8 years ago

Cool, sounds good!

On Mon, Jan 11, 2016, 1:59 AM Alex Neslusan notifications@github.com wrote:

I wanted to merge this into develop instead of into master, but I forgot about this and did the same change myself in commit 8294389 https://github.com/deadfoxygrandpa/Elm.tmLanguage/commit/8294389e53f4a47d44773071a10ec3ae41987cbb

Thanks for the idea though

— Reply to this email directly or view it on GitHub https://github.com/deadfoxygrandpa/Elm.tmLanguage/pull/68#issuecomment-170493957 .