Add ruff to pre-commit by cclauss · Pull Request #327 · cpplint/cpplint
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 302
Comments
Conversation
cclauss
requested review from
aaronliu0130 and
jayvdb
This was referenced
Mar 7, 2025aaronliu0130 previously approved these changes Mar 7, 2025
aaronliu0130
dismissed
their stale review
There's superset PRs, so I'll review those instead
jayvdb approved these changes Mar 8, 2025
Copy link
Member
aaronliu0130
commented
Mar 8, 2025
aaronliu0130 commented
Mar 8, 2025I think we should perhaps merge this PR first and then rebase the other ruff changes, sorting out their review comments afterward. @cclauss Feel free to merge this if that is indeed your intention! (That said, I feel like releasing 2.0.1 is a higher priority, though.)
cclauss
deleted the
ruff
branch
cclauss added a commit that referenced this pull request
Mar 8, 2025This 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants