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

chore(config): Update pre-commit hooks version #5

Merged
merged 1 commit into from
Nov 12, 2024
Merged

Conversation

sangshuduo
Copy link
Owner

  • Update trailing-whitespace hook to latest stable version (v5.0.0)

- Update `trailing-whitespace` hook to latest stable version (v5.0.0)
Copy link
Contributor

@mentatbot mentatbot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The version update looks good, but since it's a major version bump, we should ensure all hooks we're using are still compatible. Also, while we're updating dependencies, it might be worth checking if the typos hook (currently at v1.26.0) also has updates available.

Thanks for using MentatBot. Give comments a 👍 or 👎 to help me improve!

@@ -2,7 +2,7 @@
repos:
# Standard pre-commit hooks for generic tasks
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.4.0 # Use the latest stable version
rev: v5.0.0 # Use the latest stable version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is a major version bump (v4.4.0 -> v5.0.0), it would be helpful to include a link to the changelog or mention any breaking changes in the commit message. Also, consider updating the comment to specify 'Latest stable version as of ' to provide more context.

Suggested change
rev: v5.0.0 # Use the latest stable version
rev: v5.0.0 # Latest stable version as of 2024-01 (v5.0.0)

@sangshuduo sangshuduo merged commit a666f9f into main Nov 12, 2024
4 checks passed
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