-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Comma cleanup #35971
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
Comma cleanup #35971
Conversation
Thanks @JonathanShrek for the PR. need to run |
I've run 'black' on each file that I have changed and for some reason CI / Checks (pull_request) continues to fail. I'm just going to close the PR. Sorry for the inconvenience. I'm a first time contributor. |
No inconvenience caused - feel free to ask for help if you're stuck on something. This looks like it was on the right track, you just needed to remove the trailing commas and then run |
Thanks @MarcoGorelli I’ll give it another shot this afternoon. |
Hello @JonathanShrek! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
Comma cleanup for #35925