Skip to content

chore(deps): bump ruff from 0.15.17 to 0.16.10 - #321

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/jd/jd/sql-compare-ruff-0168/bump-ruff-0-15-17-0-16-10--480194db
Oct 2, 2026
Merged

mergify[bot] merged 1 commit into
mainfrom
devs/jd/jd/sql-compare-ruff-0168/bump-ruff-0-15-17-0-16-10--480194db

Conversation

@jd

@jd jd commented Oct 1, 2026

Copy link
Copy Markdown
Member

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

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
Copilot AI balanced review requested due to automatic review settings October 1, 2026 21:19
@mergify
mergify Bot deployed to Mergify Merge Protections October 1, 2026 21:19 Active
@mergify

mergify Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 6 merge protections satisfied — ready to merge.

Show 6 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success=Test with Python 3.10
    • check-success=Test with Python 3.11
    • check-success=Test with Python 3.12
    • check-success=Test with Python 3.13
    • check-success=Test with Python 3.14
    • check-success=semgrep

🟢 👀 Review Requirements

  • any of:
    • #approved-reviews-by>=1
    • author = dependabot[bot]

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 🔎 Reviews

  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify
mergify Bot requested a review from a team October 1, 2026 21:20

Copilot AI 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.

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
jd marked this pull request as ready for review October 2, 2026 07:16
@mergify

mergify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-10-02 08:37 UTC · Rule: default · triggered by merge protections
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-10-02 08:37 UTC · at ad22ade3ce6df1b1321f7ade4388f34d4ce0b59e · squash

This pull request spent 9 seconds in the queue, including 1 second running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Oct 2, 2026
@mergify
mergify Bot merged commit ad22ade into main Oct 2, 2026
9 checks passed
@mergify
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
@mergify mergify Bot removed the queued label Oct 2, 2026

This branch was successfully deployed

1 active deployment
Mergify Merge Protections — 3a1bfc66 Deployed Oct 1, 2026 by mergify[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants