Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate rule B904 and TRY200 are duplicate #8830

Closed
Skylion007 opened this issue Nov 23, 2023 · 2 comments · Fixed by #9680
Closed

Duplicate rule B904 and TRY200 are duplicate #8830

Skylion007 opened this issue Nov 23, 2023 · 2 comments · Fixed by #9680
Assignees
Labels
rule Implementing or modifying a lint rule
Milestone

Comments

@Skylion007
Copy link
Contributor

Rules B904 and TRY200 appear to be looking for the exact same construct (not using from within an exception block), but are separate rules. These should be deduplicated to a common implementation with an alias. We should also probably try to find duplicate rules with an automated mechanism in the future.

@charliermarsh
Copy link
Member

Yeah these do look like exact duplicates. We should remove TRY200. Thanks for noticing this.

@charliermarsh charliermarsh added the rule Implementing or modifying a lint rule label Nov 24, 2023
@Skylion007
Copy link
Contributor Author

Ah apparently, this itself is a duplicate issue: See #8736

@charliermarsh charliermarsh mentioned this issue Jan 11, 2024
13 tasks
@MichaReiser MichaReiser added this to the v0.2.0 milestone Jan 19, 2024
@zanieb zanieb self-assigned this Jan 30, 2024
@zanieb zanieb mentioned this issue Jan 30, 2024
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 a pull request may close this issue.

4 participants