-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
black #27
Comments
Det her ser mindre readable ud: https://github.com/TK-IT/web/compare/black#diff-13ac0b05b4513d9ecb1d7d8f74699946L57 |
Hvad med et diff med |
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
Jeg tænker at prøve igen om en måned eller to eller når der er et GA release. |
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. |
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
.The text was updated successfully, but these errors were encountered: