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

hlint refactoring #403

Merged
merged 1 commit into from
May 23, 2024
Merged

hlint refactoring #403

merged 1 commit into from
May 23, 2024

Conversation

flandweber
Copy link
Contributor

I just ran hlint on the codebase and applied an automatic refactoring.

Feel free to drop any part of it.

Copy link
Owner

@nmattia nmattia left a comment

Choose a reason for hiding this comment

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

I think I'd drop the uncurry and const stuff for legibility but I don't care too much. The NOINLINE is also probably overkill but I don't mind keeping it in to make hlint happy :) Thanks a lot!

@flandweber flandweber force-pushed the hlint branch 2 times, most recently from 4511385 to 464aaec Compare May 23, 2024 09:45
Was perfomed using `find -name '*.hs' -exec hlint -i "Missing NOINLINE
pragma" -i "Use uncurry" -i "Use const"  --refactor
--refactor-options="--inplace" {} \;`
@nmattia nmattia merged commit f7c5388 into nmattia:master May 23, 2024
2 checks passed
@nmattia
Copy link
Owner

nmattia commented May 23, 2024

Thanks!

@flandweber flandweber deleted the hlint branch June 10, 2024 11:17
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