Skip to content

Update precommit #93

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

Merged
merged 6 commits into from
Apr 12, 2022
Merged

Update precommit #93

merged 6 commits into from
Apr 12, 2022

Conversation

Zeitsperre
Copy link
Contributor

Pull Request Checklist:

  • This PR addresses an already opened issue (for bug fixes / features)
    • This PR fixes issue #xyz
  • Tests for the changes have been added (for bug fixes / features)
  • Documentation has been added / updated (for bug fixes / features)
  • HISTORY.rst has been updated (with summary of main changes)
  • I have added my relevant user information to AUTHORS.md
  • What kind of change does this PR introduce?:
  • Updates precommit hooks to their newest versions
  • Adds pre-commit.ci hook definitions (https://pre-commit.ci/)
  • Ran the hooks over the code base
  • Does this PR introduce a breaking change?:

No.

  • Other information:

Pre-commit.ci can manage the periodic update of hook library versions, as well as automatically apply fixes on pull requests. Someone with maintainer access simply needs to opt in to the service for the daops repo. https://results.pre-commit.ci/login/github

@cehbrecht cehbrecht self-requested a review April 12, 2022 14:57
@Zeitsperre Zeitsperre mentioned this pull request Apr 12, 2022
5 tasks
@cehbrecht
Copy link
Collaborator

@Zeitsperre needs to be re-based with master. Needed the logging PR #92 to work with clisops.

@cehbrecht cehbrecht merged commit 1dbf99e into roocs:master Apr 12, 2022
@Zeitsperre Zeitsperre deleted the update_precommit branch April 12, 2022 15:20
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