chore(deps): bump ruff from 0.15.17 to 0.16.10 - #321
Merged
mergify[bot] merged 1 commit intoOct 2, 2026
Merged
mergify[bot] merged 1 commit into
mergify[bot] merged 1 commit into
Conversation
Supersedes #318, which only moved the lockfile and left the Python 3.13 job red on `ruff check .` (21 errors, the other matrix legs cancelled behind it). Both new findings come from preview rules, which this repo opts into with `preview = true`: - `rule-codes-in-selectors`: ruff 0.16 wants rule names, not codes, in `ignore` and `per-file-ignores`. Converted with `ruff check --fix`; linter prefixes in `select` (and `"D"` in the tests ignore) are not affected. The NOTE comment now names D203/D211 by their current names too. - `property-docstring-starts-with-verb`: a property's docstring describes a value, so the 15 "Return …" / "Yield …" docstrings on properties now read as noun phrases ("The type of SQL statement."). Methods keep their imperative docstrings, as D401 still expects. The dev constraint moves to `>=0.16.8,<0.17.0`, and the lock resolves to 0.16.10, the latest 0.16 release. No other package changed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Change-Id: I480194db284d4a67a1155f23add802ded4f9f204
Contributor
Merge Protections🟢 All 6 merge protections satisfied — ready to merge. Show 6 satisfied protections🟢 🤖 Continuous Integration
🟢 👀 Review Requirements
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 Reviews
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The dependency, configuration, documentation, and lockfile changes are consistent and complete.
Review effort: Balanced
Findings: None
What changed in this PR
Updates Ruff to 0.16.10 and resolves new preview-rule findings.
Changes:
- Updates Ruff dependency and lockfile.
- Migrates rule codes to rule names.
- Rephrases property docstrings as noun phrases.
| File | Description |
|---|---|
sql_compare/__init__.py |
Updates property docstrings. |
pyproject.toml |
Updates Ruff constraint and selectors. |
poetry.lock |
Locks Ruff 0.16.10. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
jd
marked this pull request as ready for review
October 2, 2026 07:16
kozlek
approved these changes
Oct 2, 2026
Contributor
Merge Queue Status
This pull request spent 9 seconds in the queue, including 1 second running CI. Required conditions to merge
|
mergify
Bot
deleted the
devs/jd/jd/sql-compare-ruff-0168/bump-ruff-0-15-17-0-16-10--480194db
branch
October 2, 2026 08:37
This branch was successfully deployed
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
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.
Supersedes #318, which only moved the lockfile and left the Python 3.13
job red on
ruff check .(21 errors, the other matrix legs cancelledbehind it). Both new findings come from preview rules, which this repo
opts into with
preview = true:rule-codes-in-selectors: ruff 0.16 wants rule names, not codes, inignoreandper-file-ignores. Converted withruff check --fix;linter prefixes in
select(and"D"in the tests ignore) are notaffected. The NOTE comment now names D203/D211 by their current
names too.
property-docstring-starts-with-verb: a property's docstringdescribes a value, so the 15 "Return …" / "Yield …" docstrings on
properties now read as noun phrases ("The type of SQL statement.").
Methods keep their imperative docstrings, as D401 still expects.
The dev constraint moves to
>=0.16.8,<0.17.0, and the lock resolvesto 0.16.10, the latest 0.16 release. No other package changed.
Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com