Skip to content

Conversation

@amyreese
Copy link
Member

Summary

Don't flag # ruff: disable or # ruff: enable comments as commented-out code.

Test Plan

New test cases.

Issue #3711

@astral-sh-bot
Copy link

astral-sh-bot bot commented Dec 17, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@amyreese amyreese requested a review from ntBre December 17, 2025 21:58
@ntBre ntBre added the rule Implementing or modifying a lint rule label Dec 17, 2025
@amyreese amyreese changed the title [eradicate] ignore ruff:disable and ruff:enable comments in ERA001 [eradicate] ignore ruff:disable and ruff:enable comments in ERA001 Dec 18, 2025
@amyreese amyreese merged commit 9cc132f into main Dec 18, 2025
38 checks passed
@amyreese amyreese deleted the amy/era001-ruff-disable branch December 18, 2025 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rule Implementing or modifying a lint rule

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants