RIOT-OS / Release-Specs

Specification for RIOT releases and corresponding test configurations
4 stars 21 forks source link

tox: add support for black format checker #233

Closed fjmolinas closed 3 years ago

fjmolinas commented 3 years ago

Added black and applied where needed.

miri64 commented 3 years ago

Added black and applied where needed.

Can we make this a separate PR? All these extra changes make the PR harder to review.

miri64 commented 3 years ago

Please rebase!

fjmolinas commented 3 years ago

Rebased and fixed PR title and description.

fjmolinas commented 3 years ago

All green here @miri64

miri64 commented 3 years ago

Please squash!

fjmolinas commented 3 years ago

ACK. Running black . in the root dir now does not change anything.

Note that just black . would change strings, but this is skipped since is configured in tox, the equivalent would be black . --skip-string-normalization.

miri64 commented 3 years ago

Note that just black . would change strings, but this is skipped since is configured in tox, the equivalent would be black . --skip-string-normalization.

Mhhh no, this not configured in tox, but setup.cfg. So also when you call black . in that directory ;-). setup.cfg is not just for tox ;-).

miri64 commented 3 years ago

See