Skip to content

Update pre-commit hooks#87

Merged
mcflugen merged 10 commits into
developfrom
mcflugen/update-pre-commit-hooks
Jan 21, 2025
Merged

Update pre-commit hooks#87
mcflugen merged 10 commits into
developfrom
mcflugen/update-pre-commit-hooks

Conversation

@mcflugen
Copy link
Copy Markdown
Collaborator

I've updated the pre-commit hooks. In particular,

  • Bumped all versions of pre-commit hooks.
  • Remove the lint workflow. It was redundant since we use pre-commit.ci.
  • Added isort and removed reorder-python-imports pre-commit hook. Annoyingly, black and reorder-python-imports conflict on style (black adds a blank line before imports) and this seemed to be the easiest solution.

@mcflugen mcflugen force-pushed the mcflugen/update-pre-commit-hooks branch from 6da0cd9 to 3eecb44 Compare January 21, 2025 20:57
@mcflugen mcflugen merged commit 7bde339 into develop Jan 21, 2025
@mcflugen mcflugen deleted the mcflugen/update-pre-commit-hooks branch January 21, 2025 21:21
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.

1 participant