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

Use pre-commit.ci for running our pre-commit hooks #200

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

MattF-NSIDC
Copy link

@MattF-NSIDC MattF-NSIDC commented Dec 7, 2023

This is faster. pre-commit.ci will also provide some useful autoupdate features. There should be a few bot PRs soon after this is merged. Or before. I don't know :P

After this is merged, you can comment with the text "pre-commit.ci autofix" (without quotes) on its own line in any PR and the bot will come through and push a commit which fixes everything it knows how to fix. This can include, for example markdown formatting. @hazelshapiro you may find that feature useful at some point!

@MattF-NSIDC MattF-NSIDC requested a review from rmarow December 7, 2023 22:06
@MattF-NSIDC MattF-NSIDC merged commit f29bde5 into main Dec 7, 2023
2 checks passed
@MattF-NSIDC MattF-NSIDC deleted the pre-commit.ci branch December 7, 2023 22:06
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