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.

Sign up for GitHub

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

Jump to bottom

Deprecate PGH rules #8931

Closed
zanieb opened this issue Nov 30, 2023 · 0 comments · Fixed by #9680
Closed

Deprecate PGH rules #8931

zanieb opened this issue Nov 30, 2023 · 0 comments · Fixed by #9680
Labels
preview Related to preview mode features
Milestone
v0.2.0

Comments

@zanieb
Copy link
Member

zanieb commented Nov 30, 2023 •
edited
Loading

See discussion in #7506

We should deprecate this rule category gated by preview mode. Where there exists a similar rule i.e. in Bandit, we should redirect.

We can then remove the category later.
This was referenced Nov 30, 2023
Closed
Closed
@zanieb zanieb added the preview Related to preview mode features label Nov 30, 2023
@charliermarsh charliermarsh mentioned this issue Jan 11, 2024
Closed
13 tasks
@MichaReiser MichaReiser added this to the v0.2.0 milestone Jan 19, 2024
This was referenced Jan 30, 2024
Closed
Merged
Merged
zanieb added a commit that referenced this issue Feb 1, 2024
@zanieb
Redirect PHG001 to S307 and PGH002 to G010 (#9756)
8c18fc7 
Follow-up to #9754 and #9689. Alternative to #9714.
Replaces #7506 and #7507
Same ideas as #9755
Part of #8931
zanieb added a commit that referenced this issue Feb 1, 2024
@zanieb
Redirect PHG001 to S307 and PGH002 to G010 (#9756)
994514d 
Follow-up to #9754 and #9689. Alternative to #9714.
Replaces #7506 and #7507
Same ideas as #9755
Part of #8931
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
preview Related to preview mode features
Projects
None yet
Milestone
v0.2.0
Development

Successfully merging a pull request may close this issue.

Release v0.2.0 astral-sh/ruff
2 participants
@MichaReiser @zanieb