Skip to content

Conversation

@ncaq
Copy link
Contributor

@ncaq ncaq commented Mar 18, 2025

Inconsistent order and uneven indentation existed not even on a file-by-file basis,
but even within files, causing warnings.
I suggest using cabal-fmt to let the machine take care of the non-essential annoyances and not worry about them.

ncaq added 3 commits March 18, 2025 17:35
Inconsistent order and uneven indentation existed not even on a file-by-file basis,
but even within files, causing warnings.
I suggest using `cabal-fmt` to let the machine take care of the non-essential annoyances and not worry about them.
@ncaq ncaq force-pushed the format-by-cabal-fmt branch from 68a2c51 to bb66ed6 Compare March 18, 2025 08:47
Copy link
Collaborator

@parsonsmatt parsonsmatt left a comment

Choose a reason for hiding this comment

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

Thanks, this is great! If we can get this on restyled CI check then that would be fantastic.

Follow the intent of the maintainer's code review.
@parsonsmatt
Copy link
Collaborator

Wonderful, thank you!!

@parsonsmatt parsonsmatt merged commit 44e3955 into yesodweb:master Mar 19, 2025
8 checks passed
@ncaq ncaq deleted the format-by-cabal-fmt branch March 21, 2025 05:38
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