chore(pre-commit): update pre-commit hooks and add security check #4327
+838
−1,080
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Upgrade pre-commit-hooks from v4.6.0 to v5.0.0 - Add detect-private-key hook to prevent accidental private key commits - Update ruff-pre-commit from v0.7.0 to v0.12.1 The updates ensure we're using the latest versions of pre-commit hooks with bug fixes and improvements. The new detect-private-key hook adds an important security check to prevent sensitive data from being committed. Ruff update brings newer linting rules and fixes. Signed-off-by: Yihuang Yu <[email protected]>
The changes remove unnecessary string concatenation by combining split strings into single strings. This improves code readability and consistency by following Python's string formatting conventions. The modifications are purely stylistic and do not affect functionality. The changes were made to: 1. Remove unnecessary string splitting where strings could be written as single strings 2. Improve code readability by having complete strings on single lines 3. Maintain consistency with Python string formatting conventions 4. Reduce visual noise in the code from unnecessary string concatenation No functional changes were made - these are purely stylistic improvements to make the code cleaner and easier to read. The changes help maintain consistency across the test files and follow Python best practices for string formatting. Signed-off-by: Yihuang Yu <[email protected]>
|
Regular update, it fixes the style like |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The updates ensure we're using the latest versions of pre-commit hooks
with bug fixes and improvements. The new detect-private-key hook adds an
important security check to prevent sensitive data from being committed.
Ruff update brings newer linting rules and fixes.
Signed-off-by: Yihuang Yu [email protected]