Skip to content

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

Open
pre-commit-ci[bot] wants to merge 1 commit into
masterfrom
pre-commit-ci-update-config
Open

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

Conversation

@pre-commit-ci

@pre-commit-ci pre-commit-ci Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.35%. Comparing base (1313650) to head (9d6ce72).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5955      +/-   ##
==========================================
- Coverage   77.74%   75.35%   -2.40%     
==========================================
  Files        1155     1155              
  Lines      139640   139598      -42     
  Branches     5056     5051       -5     
==========================================
- Hits       108569   105199    -3370     
- Misses      29188    32775    +3587     
+ Partials     1883     1624     -259     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@njzjz-bot njzjz-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ruff 0.16.1 is a compatible patch update: the upstream mirror keeps the same hook IDs and entry points, Python 3.10+ remains supported, and both the release notes and the green CI show no formatter, CLI, or configuration regression. The inline note records the compatibility checks.\n\nCodex quota is about to reset, so I am using the remaining token budget to complete a concentrated review pass over the outstanding PRs.\n\nCoding agent: Codex\nCodex version: codex-cli 0.144.6\nModel: gpt-5.6-sol\nReasoning effort: xhigh

Comment thread .pre-commit-config.yaml Outdated
- repo: https://github.com/astral-sh/ruff-pre-commit
# Ruff version.
rev: v0.16.0
rev: v0.16.1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verified against the upstream 0.16.1 release and mirror metadata: the hook IDs and entry points are unchanged, Python 3.10+ remains supported, and this patch release contains bug/fix-safety corrections rather than a CLI or formatter breaking change. Ruff 0.16.0 and 0.16.1 both leave this tree clean, and the PR's required CI is green. No change requested here.

@pre-commit-ci
pre-commit-ci Bot force-pushed the pre-commit-ci-update-config branch from 9060af3 to 29b9f0d Compare August 10, 2026 20:49
@pre-commit-ci
pre-commit-ci Bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 11a1340 to 192e195 Compare August 24, 2026 20:49
@pre-commit-ci
pre-commit-ci Bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from 7bd8c1f to 409c2e6 Compare September 7, 2026 20:54

@njzjz-bot njzjz-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The diff is limited to pre-commit hook revisions (isort 9.0.1, Ruff 0.16.6, clang-format 23.1.0, pylint 4.0.8) with no project-code changes. I did not find a configuration/interface regression in the hook definitions. The PR is currently not mergeable against master and will need an autoupdate/rebase before merging, but the update itself is fine.

Agent: ChatGPT
Model: GPT-5.6 Sol

@njzjz-bot njzjz-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Re-reviewed the current head after the automated hook versions advanced again. The complete diff is limited to .pre-commit-config.yaml: isort 9.0.0b1→9.0.1, Ruff 0.16.0→0.16.6, clang-format 22.1.8→23.1.0, and pylint 4.0.6→4.0.8. All current build/test workflows pass, including C++, C library, Python, CUDA, CodeQL, and packaging. No blocking compatibility issue is apparent in this configuration-only update.

Agent: ChatGPT
Model: GPT-5.6 Sol
GitHub account: njzjz-bot
Reviewed head: 409c2e6
Trigger: scheduled all-PR monitoring

@pre-commit-ci
pre-commit-ci Bot force-pushed the pre-commit-ci-update-config branch from 409c2e6 to 0116114 Compare September 14, 2026 20:56

@njzjz-bot njzjz-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Re-reviewed the new pre-commit.ci head. The complete diff is limited to hook revisions in .pre-commit-config.yaml (isort 9.0.1, Ruff 0.16.7, clang-format 23.1.1, shfmt 3.14.1-1, pylint 4.0.8); I found no concrete compatibility or configuration blocker, and CUDA/build/packaging/CodeQL checks currently pass. I am not approving this head yet because the relevant Test C++ and Test Python workflows are still in progress. No additional inline issue is warranted at this point.

Agent: ChatGPT
Model: GPT-5.6 Sol
GitHub account: njzjz-bot
Reviewed head: 0116114
Trigger: scheduled all-PR monitoring

@njzjz-bot njzjz-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The two checks that were still running on the previous pass (Test C++ and Test Python) have now completed successfully. All pull-request workflow runs for this head are green, including CUDA, C++/C library builds, Python, CodeQL, and packaging. Since the diff is still limited to the reviewed pre-commit hook revisions, no blocker remains.

Agent: ChatGPT
Model: GPT-5.6 Sol
GitHub account: njzjz-bot
Reviewed head: 0116114
Trigger: scheduled all-PR monitoring

updates:
- [github.com/PyCQA/isort: 9.0.0b1 → 9.0.1](PyCQA/isort@9.0.0b1...9.0.1)
- [github.com/astral-sh/ruff-pre-commit: v0.16.0 → v0.16.8](astral-sh/ruff-pre-commit@v0.16.0...v0.16.8)
- [github.com/pre-commit/mirrors-clang-format: v22.1.8 → v23.1.1](pre-commit/mirrors-clang-format@v22.1.8...v23.1.1)
- [github.com/scop/pre-commit-shfmt: v3.13.1-1 → v3.14.1-1](scop/pre-commit-shfmt@v3.13.1-1...v3.14.1-1)
- https://github.com/pylint-dev/pylint/: v4.0.6 → v4.0.8
@pre-commit-ci
pre-commit-ci Bot force-pushed the pre-commit-ci-update-config branch from 0116114 to 9d6ce72 Compare September 21, 2026 20:45

@njzjz-bot njzjz-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NEEDS HUMAN REVIEW / waiting for exact-head CI completion.

Re-reviewed the newly updated pre-commit head. The complete diff is confined to .pre-commit-config.yaml and advances isort 9.0.0b1→9.0.1, Ruff 0.16.0→0.16.8, clang-format 22.1.8→23.1.1, shfmt 3.13.1-1→3.14.1-1, and pylint 4.0.6→4.0.8. I found no high-confidence compatibility or correctness blocker in the hook configuration itself. Exact-head Test CUDA and Build C++ are green, but Test Python, Test C++, Build C library, CodeQL, and the package/PyPI workflow are still in progress, so approval would be premature under the repository's validation requirements.

Agent: ChatGPT
Model: GPT-5.6 Sol
GitHub account: njzjz-bot
Reviewed head: 9d6ce72
Trigger: scheduled all-PR monitoring

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