Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ormolu check #550

Merged

Conversation

shayne-fletcher
Copy link
Contributor

add an ormolu check workflow covering CI.hs, ghc-lib-gen. required formatting some more source files to get the test green.

@shayne-fletcher shayne-fletcher requested a review from a team as a code owner August 1, 2024 03:36
@samuel-williams-da
Copy link
Contributor

Any chance we could get this on a git pre-commit hook? If not, at least make running the formatter easy (perhaps a shell file that passes in the patterns)

@shayne-fletcher
Copy link
Contributor Author

i think that's somewhat orthogonal to what this pr offers? i don't have much interest in pursuing this aspect further right now. perhaps you (or someone else) could look into adding that?

@samuel-williams-da
Copy link
Contributor

Sure! I'll take a look next week :)

@samuel-williams-da samuel-williams-da merged commit 2b360c7 into digital-asset:master Aug 1, 2024
26 checks passed
@shayne-fletcher shayne-fletcher deleted the ormolu-check branch August 2, 2024 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants