haskell-actions / run-ormolu

Ormolu GitHub action
Other
31 stars 6 forks source link

Don't run by default #18

Open asarkar opened 1 year ago

asarkar commented 1 year ago

It looks like uses: mrkkrp/ormolu-action@v8 immediately starts executing ormolu. This is unexpected and rarely intended. I expected the action to simply setup Ormolu, and not do anything unless told. I invoke Ormolu from a shell script so that my local build can be consistent with my GitHub CI. I also pass options to Ormolu and those should be consistent too.

I guess back to stack install ormolu for me. Too bad, because it takes forever to compile it.

On the aside, it would be helpful to see examples of passing the inputs documented here. I don't understand how to specify the pattern more than once.

mrkkrp commented 1 year ago

I added a link to an example of the syntax for multiple patterns.

asarkar commented 1 year ago

I added a link ...

Thank you. You added default, so I'm guessing for a user, it'll be like the following?

pattern: |
  **/*.hs
  **/*.hs-boot
mrkkrp commented 1 year ago

Yes, exactly.

asarkar commented 1 month ago

Can we get a setup-ormolu please, like the other setups under haskell-actions?

mrkkrp commented 1 month ago

PRs are welcome.