Skip to content

Conversation

@MaximilianAlgehed
Copy link
Collaborator

@lehins you'll like this one ;)

I don't quite know what configuration to use, I'll steal the one from ledger and see if it needs any tweaks (likely not).

closes #19

Copy link
Member

@lehins lehins left a comment

Choose a reason for hiding this comment

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

You are totally right, I do love it ❤️

Copy link
Contributor

@Soupstraw Soupstraw left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@lehins lehins left a comment

Choose a reason for hiding this comment

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

Awesome! It only needs a squash and a rebase. Other than that it is ready to go.

@MaximilianAlgehed
Copy link
Collaborator Author

Yes, but to avoid conflicts with the other open PRs I'll wait until those are done to merge this. Otherwise there is some manual faffing around with moving the fourmolu.yaml file over etc. easiest this way 😅

@MaximilianAlgehed MaximilianAlgehed enabled auto-merge (squash) November 21, 2025 08:29
@MaximilianAlgehed MaximilianAlgehed merged commit 9d0b00a into master Nov 21, 2025
10 checks passed
@MaximilianAlgehed MaximilianAlgehed deleted the fourmolu branch November 21, 2025 08:35
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.

Use consistent formatter

4 participants