search

WISVCH / dienst2

CH Dienstensysteem: administratie van leden en post
https://dienst2.ch.tudelft.nl
Other
4 stars 2 forks source link

Format all code using the Black code formatter #262

Closed praseodym closed 4 years ago

praseodym commented 4 years ago

Also includes changes from https://github.com/WISVCH/dienst2/pull/261

julian9499 commented 4 years ago

Looks good to me!

praseodym commented 4 years ago

Is er ook al een flake8 check? (of zou die er ook moeten zijn?)

Nog niet, lijkt me een goede toevoeging.

Willen we misschien ook nog iets als precommit instellen dat die al bij committen checkt of black (en flake8) checks goed zouden gaan?

Goed idee! Het wordt nu iig wel in een GitHub Action gecheckt. Kun je precommit hooks eigenlijk ook in je repo committen?

jgadelange commented 4 years ago

Volgens mij niet standaard, maar je kan met iets als: https://pre-commit.com/ het instellen (heeft ook default black/flake8 configs)

praseodym commented 4 years ago

265 en #266 aangemaakt hiervoor.