-
Notifications
You must be signed in to change notification settings - Fork 266
Encore des changements sur le README. #1293
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
Conversation
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.
Je pense qu'il convient d'ajouter un paragraphe sur la non traduction des paramètres de méthode ou fonctions (surtout si ce sont des paramètres nommés).
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
J'ai ajouté une section pour
J'ai aussi rendu le README un peu plus directif. Je m'arrête là, ça fait beaucoup de changements en une seule PR. |
Wow. 200 lignes de changement, c'est une réécriture ! Je relis ça jeudi, promis juré. |
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.
parfait 👌 quasi rien à redire à part 2-3 trucs.
Co-authored-by: Mathieu Dupuy <deronnax@gmail.com>
@deronnax merci pour la relecture. |
Arf. En écrivant Tu as mon feu vert absolu 🟢👌 |
Je suis déçu de savoir que tu penses :
Merci pour ta relecture cependant 👍 |
tiens, tu as pu fusionner malgré le "changes requested" de Christophe ? |
et j'avoue qu'en rédigeant ma phrase "il va aller vérifier", je me suis dit "en fait il le sait sûrement de tête" :D |
J'ai dû désactiver sa review oui. Je me permets ce genre de choses quand j'ai pris en compte toutes ses remarques même si ça fait un peu forceur. |
make todo
plutôt quepotodo
(remonté par @deronnax sur idées prochaines améliorations pour le README #1289) ;Les changements sont visibles ici.