search

TK-IT / meta

Diskussioner, referater, dokumentation mm.
0 stars 0 forks source link

black #27

Open neic opened 6 years ago

neic commented 6 years ago

Rav og jeg har snakket om at bruge bruge black på vores repos. Jeg har prøvet at kører den nyeste version (18.05b1) på web. Her er en diff. De fleste er ændringer fra single quote til double quotes som også er et kontroversielt issue i black.

Black er i beta og der kommer stadig en rimelig jevn strøm af bugs.

Jeg vil foreslå at vi venter til det er mere modent inden vi tilføjer det. Så vi ikke får flere end et commit der ændre store dele af kodebasen og skal håndteres af når man i fremtiden laver git blame.

tyilo commented 6 years ago

Det her ser mindre readable ud: https://github.com/TK-IT/web/compare/black#diff-13ac0b05b4513d9ecb1d7d8f74699946L57

tyilo commented 6 years ago

Hvad med et diff med --skip-string-normalization?

neic commented 6 years ago

Jeg er enig at der steder hvor det ikke er optimalt. Ideen er dog at istedet for at vi, som udviklere af et projekt ,skal blive enige om hvilken kodestil vi skal bruge, kan communityet omkring black tage diskussionerne. Vi kan på den måde bytte diskussioner og/eller fussy kodestil med nogle få linjer kode som ikke er optimalt stylet.

--skip-string-normalization er ikke i en released version endnu og er også en af grundene til at vente lidt.

Jeg tænker at prøve igen om en måned eller to eller når der er et GA release.

Mortal commented 6 years ago

Hvis jeg skulle deltage i udviklingen af et projekt der bruger black, ville jeg nok ikke have noget imod det - jeg tror kun jeg er fintfølende fordi det er vores projekt (og ikke nogle andres projekt) det drejer sig om.