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

Consider removing PD901 #7710

Open
charliermarsh opened this issue Sep 29, 2023 · 7 comments · Fixed by #9680
Open

Consider removing PD901 #7710

charliermarsh opened this issue Sep 29, 2023 · 7 comments · Fixed by #9680
Labels
breaking Breaking API change
Milestone

Comments

@charliermarsh
Copy link
Member

charliermarsh commented Sep 29, 2023

PD901 disallows df as a variable name. This is an opinionated rule in pandas-vet so is typically only enabled via explicit opt-in. But we don't have that behavior for 900-level rules, so it's always enabled if you enable PD. The rule is overly strict for most cases, so I think it's a net-negative to have it in Ruff given our behavior.

@charliermarsh charliermarsh added the needs-decision Awaiting a decision from a maintainer label Sep 29, 2023
@dhruvmanila
Copy link
Member

I'm in favor of this 👍

@charliermarsh charliermarsh mentioned this issue Jan 11, 2024
13 tasks
@MichaReiser MichaReiser added this to the v0.2.0 milestone Jan 19, 2024
@zanieb
Copy link
Member

zanieb commented Jan 30, 2024

@charliermarsh can you fill this issue in?

@kdebrab
Copy link

kdebrab commented Sep 27, 2024

AFAICT, it seems like rule PD901 has not been deprecated?

@dhruvmanila
Copy link
Member

Yeah, it wasn't removed. cc @charliermarsh

@charliermarsh
Copy link
Member Author

I don't think I closed it! But I'll re-open.

@charliermarsh charliermarsh reopened this Oct 1, 2024
@zanieb
Copy link
Member

zanieb commented Oct 1, 2024

Can you fill in the context on why we should consider removing the rule?

@charliermarsh
Copy link
Member Author

Sure.

@charliermarsh charliermarsh added breaking Breaking API change and removed needs-decision Awaiting a decision from a maintainer labels Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking API change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants