-
Notifications
You must be signed in to change notification settings - Fork 1
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
14 colorisation des nuages de point étroits largeur ou hauteur 02m #15
The head ref may contain hidden characters: "14-colorisation-des-nuages-de-point-\u00E9troits-largeur-ou-hauteur-02m"
14 colorisation des nuages de point étroits largeur ou hauteur 02m #15
Conversation
Ca me parait ok, mais je préfère attendre le retour de @gliegard là-dessus avant le merge, je pense qu'il a plus de recul que moi sur cette partie du code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pas de problème pour les modifications dans le code. Merci pour cette amélioration.
Pas contre, si tu veux vraiment faire du lint, fait le dans un commit à part.
Merci ! (My bad pour le linting ; j'ai reconfiguré mon VS Code pour qu'il prenne la config du projet quand il y en aura une dans pyproject.toml, ce qui semble une bonne idée :)) |
J'ai rajouté une config black ici : #18 |
9a8fd43
to
36268a8
Compare
@leavauchier C'est rebasé sur dev, et j'ai appliqué black avec la bonne config sur color.py et test_color.py. Au passage j'ai remarqué qu'il faudrait appliquer la config de black sur l'ensemble des tests :) |
e5c0b74
to
2dd8467
Compare
proposition de solution pour les deux soucis évoqués dans #14