Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Reformat repository to clean up failing CI step #3131

Closed
wants to merge 3 commits into from

Conversation

drsooch
Copy link
Collaborator

@drsooch drsooch commented Sep 1, 2022

The pre-commit action appears to be failing. I did this in a separate PR but that didn't seem to catch everything. This PR should clean up all formatting problems so we can stop failing builds.

@drsooch
Copy link
Collaborator Author

drsooch commented Sep 2, 2022

@wz1000 to get stylish-haskell to parse properly, I've had to move some of the CPP blocks around. I tried to verify everything's good, but if you wouldn't mind double-checking.

@kokobd
Copy link
Collaborator

kokobd commented Sep 5, 2022

pre-commit as a CI workflow was added one month ago. By default, it runs on --all-files. Maybe we can simply change its args to let it only run on changed files, like what we did before locally?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants