chore(deps-dev): update ruff requirement from <0.16,>=0.15 to >=0.15,<0.17 in the python-dependencies group across 1 directory - #25
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0430b5c016
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "pytest>=8,<10", | ||
| "PyYAML>=6,<7", | ||
| "ruff>=0.15,<0.16", | ||
| "ruff>=0.15,<0.17", |
There was a problem hiding this comment.
Bump the synchronized plugin version
Because this change modifies pyproject.toml, it falls under the repository's release-relevant paths and must include the synchronized version bump in plugin.yaml, pyproject.toml, and hermes_local_knowledge/__init__.py. As submitted, scripts/check_version_policy.py --base-ref 81b5c8ea125765e1b2341bfb57538c5eaaaa659d^ --head-ref 81b5c8ea125765e1b2341bfb57538c5eaaaa659d reports Release-relevant files changed without a plugin version bump, so the required version-policy gate will reject this dependency update.
Useful? React with 👍 / 👎.
| "pytest>=8,<10", | ||
| "PyYAML>=6,<7", | ||
| "ruff>=0.15,<0.16", | ||
| "ruff>=0.15,<0.17", |
There was a problem hiding this comment.
Keep Ruff defaults pinned before allowing 0.16
The CI job installs .[test] and then runs python -m ruff check . (.github/workflows/ci.yml:36-52), but this range lets pip select Ruff 0.16, whose migration notes say the default rules expand from 59 to 413 (https://astral.sh/blog/ruff-v0.16.0). This repo has no [tool.ruff.lint] select config, and the current tree already fails rules that are in Ruff's new default set (python -m ruff check --select BLE001,UP035,I001 . reports 47 errors), so after the version-policy issue is fixed the lint step will still fail unless the old defaults are explicitly selected or the new diagnostics are fixed/ignored.
Useful? React with 👍 / 👎.
0430b5c to
56c072b
Compare
Updates the requirements on [ruff](https://github.com/astral-sh/ruff) to permit the latest version. Updates `ruff` to 0.16.0 - [Release notes](https://github.com/astral-sh/ruff/releases) - [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md) - [Commits](astral-sh/ruff@0.15.0...0.16.0) --- updated-dependencies: - dependency-name: ruff dependency-version: 0.16.0 dependency-type: direct:development dependency-group: python-dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
56c072b to
ac36b5e
Compare
|
Superseded by #26, which rebases the dependency update onto current main, preserves Ruff’s pre-0.16 default lint rules explicitly, and includes the repository-required synchronized version bump and release notes. |
|
This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests. To ignore these dependencies, configure ignore rules in dependabot.yml |
Rebasing might not happen immediately, so don't worry if this takes some time.
Note: if you make any changes to this PR yourself, they will take precedence over the rebase.
Updates the requirements on ruff to permit the latest version.
Updates
ruffto 0.16.0Release notes
Sourced from ruff's releases.
... (truncated)
Changelog
Sourced from ruff's changelog.
... (truncated)
Commits
a2635fdBump 0.16.0 (#27136)3433449[ty] Reuse full call diagnostics for implicit setter calls (#27115)2240070Reflectruff: ignoreand--add-ignorestabilization in documentation (#27...17ef711Stabilize--add-ignore(#27125)ef912bbAdd newly stabilized rules to defaults (#27055)b30f040Stabilize new default rules (#27035)bcd70c5Exclude Markdown files fromformat-devruns (#27052)87e51e2Fixformat --checkspans for syntax errors (#27045)afe2723[flake8-gettext] Stabilize qualified-name and built-in binding resolution (...a9702d8[flake8-bandit] Stabilize string literal binding resolution (S310) (#26944)