Skip to content

Conversation

@Borda
Copy link
Contributor

@Borda Borda commented Jan 13, 2026

This pull request updates the .pre-commit-config.yaml file to use newer versions of several pre-commit hooks, ensuring the codebase benefits from the latest features and bug fixes. It also switches the Ruff hook to a newer recommended id.

Pre-commit hook version upgrades:

  • Upgraded pre-commit-hooks from v5.0.0 to v6.0.0 for improved linting and formatting checks.
  • Upgraded ruff-pre-commit from v0.7.0 to v0.14.11 and changed the hook id from ruff to ruff-check to align with the latest configuration recommendations.
  • Upgraded pycln from v2.5.0 to v2.6.0 for more robust import cleaning.

Copilot AI review requested due to automatic review settings January 13, 2026 08:46
Signed-off-by: jirka <jirka.borovec@seznam.cz>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 13, 2026

📝 Walkthrough

Walkthrough

Updated .pre-commit-config.yaml with version bumps for three pre-commit hooks: pre-commit-hooks (v5.0.0 → v6.0.0), ruff-pre-commit (v0.7.0 → v0.14.11), and pycln (v2.5.0 → v2.6.0). Changed ruff hook identifier from "ruff" to "ruff-check" with no argument modifications.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive Description covers the main changes and purpose well, but doesn't follow the provided template structure with required sections like 'Types of changes' checkboxes. Add the 'Types of changes' section from the template to indicate which type of change this is (e.g., non-breaking change).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed Title clearly and concisely summarizes the main change: updating pre-commit hooks to newer versions.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 57fdd59 and 84bfaa0.

📒 Files selected for processing (1)
  • .pre-commit-config.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (12)
  • GitHub Check: min-dep-pytorch (2.6.0)
  • GitHub Check: min-dep-pytorch (2.7.1)
  • GitHub Check: min-dep-pytorch (2.8.0)
  • GitHub Check: min-dep-pytorch (2.5.1)
  • GitHub Check: min-dep-os (macOS-latest)
  • GitHub Check: min-dep-os (windows-latest)
  • GitHub Check: flake8-py3 (pytype)
  • GitHub Check: quick-py3 (macOS-latest)
  • GitHub Check: packaging
  • GitHub Check: quick-py3 (windows-latest)
  • GitHub Check: flake8-py3 (mypy)
  • GitHub Check: quick-py3 (ubuntu-latest)
🔇 Additional comments (3)
.pre-commit-config.yaml (3)

11-12: LGTM — version bump is valid.

pre-commit-hooks v6.0.0 now requires Python >= 3.9 and removed check-byte-order-marker and fix-encoding-pragma. None of the removed hooks are used here.


29-33: LGTM — aligns with Ruff's official pre-commit config.

v0.14.11 with ruff-check matches the current recommended configuration from Ruff.


57-58: pycln v2.6.0 is a valid git tag. No concerns with the version update.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates three pre-commit hooks to their latest versions and changes the Ruff hook ID to follow current recommendations. The updates ensure the project benefits from the latest bug fixes and features in these code quality tools.

Changes:

  • Upgraded pre-commit-hooks from v5.0.0 to v6.0.0
  • Upgraded ruff-pre-commit from v0.7.0 to v0.14.11 and changed hook ID from ruff to ruff-check
  • Upgraded pycln from v2.5.0 to v2.6.0
Comments suppressed due to low confidence (1)

.pre-commit-config.yaml:40

  • An extra blank line has been added here. The file already has a blank line at line 39, so this creates consecutive blank lines which is inconsistent with the rest of the file's formatting. Consider removing this additional blank line.
  - repo: https://github.com/asottile/yesqa

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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