Skip to content

Introduce ormolu as the Haskell formatter#285

Merged
Y-Nak merged 4 commits into
argotorg:mainfrom
Y-Nak:introduce-ormolu
Feb 6, 2026
Merged

Introduce ormolu as the Haskell formatter#285
Y-Nak merged 4 commits into
argotorg:mainfrom
Y-Nak:introduce-ormolu

Conversation

@Y-Nak

@Y-Nak Y-Nak commented Dec 10, 2025

Copy link
Copy Markdown
Member

Summary

  • Add ormolu format check to CI via nix flake check
  • Format all existing Haskell files with ormolu
  • Add ormolu to dev shell for local formatting
  • Update README with formatting commands

@Y-Nak

Y-Nak commented Dec 10, 2025

Copy link
Copy Markdown
Member Author

If you have a favorite formatter, please let me know. I'll modify this PR.

@rodrigogribeiro rodrigogribeiro left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks!

@mbenke

mbenke commented Dec 11, 2025

Copy link
Copy Markdown
Collaborator

One downside would be a flurry of merge conflicts for all pending PRs. So I would postpone merging this PR aftger we merge critical PRs for ERC-20.

@Y-Nak

Y-Nak commented Dec 11, 2025

Copy link
Copy Markdown
Member Author

I would postpone merging this PR aftger we merge critical PRs for ERC-20.

Sounds reasonable. Also, I'd like to hear an opinion from @d-xo.

@Y-Nak Y-Nak requested a review from mbenke February 5, 2026 14:46

@mbenke mbenke left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@Y-Nak Y-Nak merged commit fd52209 into argotorg:main Feb 6, 2026
3 checks passed
@Y-Nak Y-Nak deleted the introduce-ormolu branch February 6, 2026 08:19
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.

3 participants