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
Black suddenly wants to add a bunch of commas? #15254
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer it if black didn't do this, but all these changes are fine and consistent with our code style.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving based on reply on psf/black#3859 . Thanks for tracking it down and separating it out to a different PR!
devdeps failure is unrelated.
|
Description
As part of transitioning from using the
setup.cfg
file to using thepyproject.toml
file for project configuration in #15247,black
for reasons which are unclear to me decided that it wants to alter other code despite the fact that no alterations to the configuration for black have been made. @astrofrog in #15247 (comment) requested that these changes be separated from the main changes proposed by #15247. This PR makes all these automated changes proposed byblack
as the triggered by #15247, as these changes are apparently fine without the main changes from #15247.