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

Rename too-many-positional{→-arguments} #12619

Closed
flying-sheep opened this issue Aug 2, 2024 · 1 comment · Fixed by #12905
Closed

Rename too-many-positional{→-arguments} #12619

flying-sheep opened this issue Aug 2, 2024 · 1 comment · Fixed by #12905
Labels
good first issue Good for newcomers help wanted Contributions especially welcome

Comments

@flying-sheep
Copy link
Contributor

In pylint-dev/pylint#9842, they renamed too-many-positional to too-many-positional-arguments and implemented it.

If we at some point implement support for full names in addition to the codes we support already, this will affect us too.

PS: What’s the schedule for this rule coming out of preview?

@AlexWaygood
Copy link
Member

Annoyingly, these are strictly-speaking parameters rather than arguments... but the new name seems strictly better than the old one, and it's useful to have the same name as pylint. PR welcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Contributions especially welcome
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants