Move to black code formatting

PSLmodels / taxdata

The TaxData project prepares microdata for use with the Tax-Calculator microsimulation project.

http://pslmodels.github.io/taxdata/

Other

21 stars 30 forks source link

Move to black code formatting #424

Closed jdebacker closed 1 year ago

jdebacker commented 1 year ago

This PR addresses Issue #422 by moving to code formatting by black.

jdebacker commented 1 year ago

@bodiyang I think this PR is ready for review.

I ran black to format the .py files (surprisingly few (just 6) needed any modification
I added a GH Action to check that the formatting is consistent with black on each PR/push
I removed the pre-commit checks. These were giving me trouble and with this PR the code formatting (which is all that was checked there) will be verified on each PR/push.
Adds black to the taxdata-dev environment

Let me know if anything else is needed before merging.

bodiyang commented 1 year ago

@bodiyang I think this PR is ready for review.

I ran black to format the .py files (surprisingly few (just 6) needed any modification

I added a GH Action to check that the formatting is consistent with black on each PR/push

I removed the pre-commit checks. These were giving me trouble and with this PR the code formatting (which is all that was checked there) will be verified on each PR/push.

Adds black to the taxdata-dev environment

Let me know if anything else is needed before merging.

Thanks Jason, this PR looks good to merge

jdebacker commented 1 year ago

@bodiyang Thanks for reviewing!

Do you or @andersonfrailey want to merge? I do not have merge privileges here. Thanks!