search

CCSDSPy / ccsdspy

I/O interface and utilities for CCSDS binary spacecraft data in Python. Library used in flight missions at NASA, NOAA, and SWRI
https://ccsdspy.org
BSD 3-Clause "New" or "Revised" License
75 stars 18 forks source link

Black reformatting with default line length #31

Closed ehsteve closed 1 year ago

ehsteve commented 1 year ago

Just wanted to show what changes black would make to the codebase if it were to be used. Looking over it myself, it seems like the changes are fairly benign and lead to more readable code. Just a reminder that the advantage of using black is that code formatting is now specified and diffs should be cleaner. It is also easier for contributors if you just ask them to run black on their code to keep it nice and formatted.

ddasilva commented 1 year ago

Thank you, this is reasonable. Can you add a black check to the CI pipeline (.github/workflows) so that regressions trigger a notification?

ddasilva commented 1 year ago

Looks good, thanks!