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.

Sign up for GitHub

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

Jump to bottom

Update pre commit hooks #133

Merged
merged 11 commits into from
Nov 1, 2021
Merged

Update pre commit hooks #133

merged 11 commits into from
Nov 1, 2021
+135 −102

Conversation

LilSpazJoekp
Copy link
Member

@LilSpazJoekp LilSpazJoekp commented Nov 1, 2021 •
edited
Loading

LilSpazJoekp and others added 10 commits November 1, 2021 13:19
@LilSpazJoekp
Consolidate flake8 config to .flake8
67261c3 
(cherry picked from commit 4cf4464b89d5d62d350ce1fd2b414decc59ac2c4)
@LilSpazJoekp
Update black config
51dd07d 
(cherry picked from commit d9aaead9505d1e50c395010210c9f89e0c1d79ff)
@LilSpazJoekp
Move docstrfmt config to pyproject.toml
175ab18 
(cherry picked from commit e40a958e68c12fe48b09aacf115adbfa11d73016)
@LilSpazJoekp
Add pre_push tasks to .pre-commit-config.yaml
1b3a5aa 
(cherry picked from commit 6eb661b6b8d6b1374c7e994c09994fd6ae240326)
@LilSpazJoekp
Add workflow for keeping pre-commit hooks updated
28fd979 
(cherry picked from commit 9fddbe1a7217a84987bdcab0e4b257dd7a0c85e6)
@LilSpazJoekp
Fix end of files
2761de1
@LilSpazJoekp
Fix flake8 complaining
44da3e4
@CAM-Gerlach @LilSpazJoekp
Update hook versions and tweak config to avoid crashes & long run times
013e76d 
(cherry picked from commit 8e1d1ca19442bfad62000e1693ddf1c3ecbf7c5a)
@CAM-Gerlach @LilSpazJoekp
Use pre-commit to simplify pre-push script and avoid duplication
66fb7a1 
(cherry picked from commit 18098843b849905057b76c530a9117cf83f6bd81)
@CAM-Gerlach @LilSpazJoekp
Unify CIs to use common pre-commit action to run linters
6f6b967 
(cherry picked from commit c0bc88269d61a7bea2252c4d5e07a84c81c5b75a)
@LilSpazJoekp LilSpazJoekp force-pushed the update_pre_commit_hooks branch from e048aee to 6f6b967 Compare November 1, 2021 18:59
@CAM-Gerlach @LilSpazJoekp
Update both contributing guides to reflect addition of pre-commit hooks
1d0b118 
(cherry picked from commit a469babb2603fb6265cc4253b864eaae17649604)
@LilSpazJoekp LilSpazJoekp merged commit b2db35f into master Nov 1, 2021
@LilSpazJoekp LilSpazJoekp deleted the update_pre_commit_hooks branch November 1, 2021 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LilSpazJoekp @CAM-Gerlach