Skip to content

[refurb] Make fix unsafe if it deletes comments (FURB116)#22681

Merged
ntBre merged 4 commits intoastral-sh:mainfrom
chirizxc:FURB116
Jan 19, 2026
Merged

[refurb] Make fix unsafe if it deletes comments (FURB116)#22681
ntBre merged 4 commits intoastral-sh:mainfrom
chirizxc:FURB116

Conversation

@chirizxc
Copy link
Contributor

Summary

Test Plan

@astral-sh-bot
Copy link

astral-sh-bot bot commented Jan 18, 2026

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Just a couple of nits

@ntBre ntBre added the fixes Related to suggested fixes for violations label Jan 19, 2026
chirizxc and others added 3 commits January 19, 2026 19:47
…t.rs

Co-authored-by: Brent Westbrook <36778786+ntBre@users.noreply.github.com>
…t.rs

Co-authored-by: Brent Westbrook <36778786+ntBre@users.noreply.github.com>
Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@ntBre ntBre enabled auto-merge (squash) January 19, 2026 16:55
@ntBre ntBre merged commit 336413b into astral-sh:main Jan 19, 2026
40 checks passed
@chirizxc chirizxc deleted the FURB116 branch January 19, 2026 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fixes Related to suggested fixes for violations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants