-
Notifications
You must be signed in to change notification settings - Fork 697
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
add Makefile targets for fix-whitespace #10376
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic
I wish any new target in the makefile came with two things:
|
This is a reasonable request yes |
b493d98
to
ca2cbc4
Compare
`make whitespace` now runs `fix-whitespace --check --verbose` and `make fix-whitespace` runs `fix-whitespace --verbose`.
ca2cbc4
to
2858dc0
Compare
@Mergifyio backport 3.14 |
✅ Backports have been created
|
add Makefile targets for fix-whitespace (backport #10376)
@Mergifyio backport 3.12 |
✅ Backports have been created
|
add Makefile targets for fix-whitespace (backport #10376)
make whitespace
now runsfix-whitespace --check --verbose
andmake fix-whitespace
runsfix-whitespace --verbose
. You will need to installfix-whitespace
yourself, but that's just acabal install
away.This is another step in the direction of fixing #10263.
Template B: This PR does not modify behaviour or interface
E.g. the PR only touches documentation or tests, does refactorings, etc.
Include the following checklist in your PR: