style(ruff): disable PLW1641
Metadata
Current evaluation
Merged PR #409 as commit 095cb0a, disabling Ruff rule PLW1641 (eq-without-hash); all CI checks passed.
Resolution: closed resolved
Reason: The configuration change was merged through PR #409 in commit 095cb0a after two approvals, with all reported CI checks passing.
Evaluated by: mr-cal-bot
No scores available.
Issue body
https://docs.astral.sh/ruff/rules/eq-without-hash/
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #409 as commit 095cb0a, disabling Ruff rule PLW1641 (eq-without-hash); all CI checks passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #409 as commit 095cb0a, disabling Ruff PLW1641 (eq-without-hash) in configuration. The one-line change received two approvals and passed all listed CI checks. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after two approvals and passing CI checks. Disables the Ruff PLW1641 (eq-without-hash) lint rule via a single configuration change. Related lint fixes were handled in PRs #410 and #411. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the change to disable the Ruff PLW1641 lint rule. Related lint fixes were addressed in separate pull requests #410 and #411. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #415 style: fix ruff 0.16 warning | craft-store | merged | Merged PR #415 as commit b24d684, adding a directive to ignore Ruff 0.16's new warning; all CI checks passed. | |
| #351 linters: add PLR2004 to ruff ignore | craft-parts | merged | Merged PR #351 added PLR2004 to Ruff's test ignores, suppressing over 600 magic-value warnings. Landed as commit 517f95d8. | |
| #907 style: adjust linting rules for ruff 0.12 | rockcraft | merged | Merged into main as 186cb0fb, updating Ruff 0.12 settings to ignore PLC0415 and avoid unnecessary noqa comments. | |
| #6378 style: ruff 0.16 | snapcraft | merged | Housekeeping PR merged to main: ignore new ruff 0.16 rule PLR0917, bump cryptography for a CVE, and drop an outdated maturin constraint. Approved by 2 reviewers. | |
| #1288 style: fix ruff PTH211 errors | craft-parts | merged | Merged as commit 13382674: PR #1288 fixed Ruff 0.13 PTH211 errors across eight files (+29/-30), with all CI checks passing. | |
| #139 style: stricter ruff linters | starbase | merged | PR #139 was merged as commit a48d590, adding stricter Ruff lint rules across six files while retaining 100% test coverage. | |
| #301 lint: fix ruff warning PLC1901 | craft-providers | merged | Merged PR #301 (commit d547a8f) by maintainer mr-cal replaced `string == ""` checks with `not string` in 4 files (+4/-5), clearing ruff PLC1901 lint warnings. |