Skip to content

[flake8-errmsg] Avoid shadowing existing msg in fix for EM101#24363

Merged
dylwil3 merged 3 commits intoastral-sh:mainfrom
dylwil3:fresh-bindings
Apr 2, 2026
Merged

[flake8-errmsg] Avoid shadowing existing msg in fix for EM101#24363
dylwil3 merged 3 commits intoastral-sh:mainfrom
dylwil3:fresh-bindings

Conversation

@dylwil3
Copy link
Copy Markdown
Collaborator

@dylwil3 dylwil3 commented Apr 2, 2026

Closes #24335

As suggested, we use the new fresh_binding helper from #24316

Note that the issue with this fix was already brought up in the discussion in #9052 (see also #9059), where it was decided that it was okay because the fix was already marked as unsafe.

@dylwil3 dylwil3 added bug Something isn't working fixes Related to suggested fixes for violations labels Apr 2, 2026
@astral-sh-bot astral-sh-bot bot requested a review from ntBre April 2, 2026 12:58
@astral-sh-bot
Copy link
Copy Markdown

astral-sh-bot bot commented Apr 2, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@dylwil3 dylwil3 merged commit a0356f1 into astral-sh:main Apr 2, 2026
42 checks passed
carljm added a commit that referenced this pull request Apr 2, 2026
* main:
  Add a "release-gate" step to the release workflow (#24365)
  Disallow starred expressions as values of starred expressions (#24280)
  [`pyupgrade`] Ignore strings with string-only escapes (`UP012`) (#16058)
  [ty] Improve consistency and quality of diagnostics relating to invalid type forms (#24325)
  [flake8-type-checking] Clarify import cycle wording for TC001/TC002/TC003 (#24322)
  [`flake8-errmsg`] Avoid shadowing existing `msg` in fix for `EM101` (#24363)
  `RUF072`: skip formfeeds on dedent (#24308)
  Replace unmaintained `unic-ucd-category` crate with `icu_properties` (#24344)
  [ty] Replace markdown hard line breaks in snapshot tests (#24361)
  [ty] Move snapshot for code action test with trailing whitespace to external file (#24359)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fixes Related to suggested fixes for violations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

EM101 fix rebinds or shadows a variable named msg

3 participants