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

Reformat code using black 24.4.2 with -t py39 #586

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FedericoCeratto
Copy link
Contributor

Hello and thanks for developing this tool.
In the past the use of Black to format the code was proposed in #125
I would like to propose this PR to reformat the project using Black 24.4.2 with -t py39 and no other option.
A next step, if there is interest, could be to document the use of Black and recommend running it before commits.

@ianmcorvidae
Copy link
Contributor

I know that black is being used fairly broadly in Python these days, but I'm not super familiar with the different presets and options it provides. If you're able to, could you explain the choice of -t py39?

Also, ff we're going to do this, I would also definitely like it to end up in the GitHub Actions as part of CI in whatever fashion is appropriate for the tool (perhaps simply a check that the code is unchanged by running the tool, but there may be preferred methods). Do you have any insight on that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants