FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[pre-commit.ci] pre-commit autoupdate by pre-commit-ci[bot] · Pull Request #460 · cpplint/cpplint · GitHub

[pre-commit.ci] pre-commit autoupdate - #460

Draft
pre-commit-ci[bot] wants to merge 1 commit into
developfrom
pre-commit-ci-update-config
Draft

[pre-commit.ci] pre-commit autoupdate#460
pre-commit-ci[bot] wants to merge 1 commit into
developfrom
pre-commit-ci-update-config

Conversation

pre-commit-ci Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

updates:
- [github.com/codespell-project/codespell: v2.4.2 → v2.4.3](codespell-project/codespell@v2.4.2...v2.4.3)
- [github.com/astral-sh/ruff-pre-commit: v0.15.20 → v0.16.1](astral-sh/ruff-pre-commit@v0.15.20...v0.16.1)
- [github.com/tox-dev/pyproject-fmt: v2.25.1 → v2.26.0](tox-dev/pyproject-fmt@v2.25.1...v2.26.0)
aaronliu0130 marked this pull request as draft August 3, 2026 22:32

yangfan-yf-yf left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Ruff 0.16.1 stabilizes ISC004, which is selected by this repository's ISC rule group. I reproduced the failing hook at 0bdcc38: it reports 12 existing unparenthesized string concatenations in cpplint_unittest.py, while the same tree passes with Ruff 0.15.20. Most are intentional wrapped strings that need explicit parentheses; cpplint_unittest.py:4000-4001 appears to intend a separate empty source line and may need a comma instead.

All hosted build and test jobs pass, so this lint migration is the remaining blocker. This draft should stay unapproved until those cases are resolved and the updated hook is green.

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
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


Back | FazBrowse Home | New Git URL