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

Pyupgrade can now support directories as input #344

Closed
wants to merge 3 commits into from

Conversation

saroad2
Copy link

@saroad2 saroad2 commented Oct 16, 2020

Pyupgrade should be able to handle directories as inputs, like any other static code analysis tools (pylint, flake8, mypy, black, etc.)

Added this ability 😃.

@MarcoGorelli
Copy link
Contributor

This looks like a duplicate of #150 #46 #339

@saroad2
Copy link
Author

saroad2 commented Oct 16, 2020

Hey @MarcoGorelli ,
I figured out that even though a lot of people have opened issues about recursive Pyupgrade, nobody actually did a PR to implement it. I thought that if I "just do it", than maybe the maintainers of Pyupgrade will consider merging it.
It does not damage any of the current functionalities, just adding a new one.

Moreover, I should state that the only reason I don't integrate Pyupgrade with my own projects, especially Statue, is because of the absence of this ability. I hope that now that I've shown that this is possible, you'd reconsider.

@asottile asottile closed this Oct 16, 2020
Repository owner locked as resolved and limited conversation to collaborators Oct 16, 2020
@saroad2 saroad2 deleted the support_directories branch October 16, 2020 16:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants