Contemplate running the black autoformatter on this code

robotpy / robotpy-cppheaderparser

DEPRECATED: use cxxheaderparser instead

Other

123 stars 39 forks source link

Contemplate running the black autoformatter on this code #8

Closed virtuald closed 4 years ago

virtuald commented 5 years ago

On the one hand, all other RobotPy projects are autoformatted with black (or are moving in that direction), and it makes it so we don't have to worry so much about style. On the other hand, if there are ever changes or pull requests upstream, we probably want to be able to merge those in without significant effort?

Undecided on this for now.

virtuald commented 4 years ago

I've decided that the code is really annoying to deal with as it currently is formatted. Furthermore, if there are upstream changes, we can just format that code with black, then merge the changes in, and it should be fairly straightforward.