Skip to content

Conversation

@sangshuduo
Copy link
Owner

  • Updated the version of the typos hook from v1.28.1 to v1.
  • This change simplifies the versioning to the latest release.
  • Ensures that we are using the most recent features and fixes available.

- Update the `typos` hook in the `.pre-commit-config.yaml` file
- Change the `rev` value from `v1.26.0` to `v1.28.1`
- Updated the version of the typos hook from v1.28.1 to v1.
- This change simplifies the versioning to the latest release.
- Ensures that we are using the most recent features and fixes available.
@mentatbot
Copy link
Contributor

mentatbot bot commented Apr 17, 2025

I'm getting to work. See my steps and track my cost here.

  • Wake on any new activity.
    I always see new activity while actively working. When paused, I'll only respond when tagged unless this box is checked.

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.

This is a straightforward update to the pre-commit configuration that changes the typos hook version from a specific version (v1.26.0) to just the major version (v1).

Benefits:

  • Simplifies version management
  • Automatically picks up new features and bug fixes within the v1.x line
  • Reduces the need for frequent version updates in the configuration

Considerations:

  • Using v1 makes builds slightly less deterministic compared to pinning a specific version
  • However, this is generally acceptable for development tools like pre-commit hooks when the tool follows semantic versioning (which typos does)

The change looks good to me. Similar versioning strategies are used for the other hooks in this file (like pre-commit-hooks at v5.0.0). This approach strikes a good balance between staying up-to-date and maintaining stability.

@sangshuduo sangshuduo merged commit 7f1fc75 into main Apr 17, 2025
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.

2 participants