Skip to content

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

Merged
merged 20 commits into from
May 22, 2020
Merged

Encore des changements sur le README. #1293

merged 20 commits into from
May 22, 2020

Conversation

awecx
Copy link

@awecx awecx commented May 17, 2020

Les changements sont visibles ici.

Copy link
Contributor

@christopheNan christopheNan left a 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).

@awecx awecx requested a review from christopheNan May 18, 2020 19:46
@awecx
Copy link
Author

awecx commented May 18, 2020

J'ai ajouté une section pour

  • les paramètres de fonction (comme remonté par @christopheNan ) ;
  • les liens hypertextes ;
  • les balises.

J'ai aussi rendu le README un peu plus directif.

Je m'arrête là, ça fait beaucoup de changements en une seule PR.

@awecx awecx requested a review from deronnax May 18, 2020 19:48
@deronnax
Copy link
Collaborator

Wow. 200 lignes de changement, c'est une réécriture ! Je relis ça jeudi, promis juré.

@deronnax deronnax self-assigned this May 18, 2020
Copy link
Collaborator

@deronnax deronnax left a 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.

awecx and others added 3 commits May 22, 2020 21:22
@awecx
Copy link
Author

awecx commented May 22, 2020

@deronnax merci pour la relecture.
Tes remarques sont pertinentes, je les ai prises en compte (avec « join» plutôt que « wait » pour être cohérent jusqu'au bout).

@awecx awecx requested a review from deronnax May 22, 2020 19:42
@deronnax
Copy link
Collaborator

Arf. En écrivant thread.join, je me suis dit "si c'est un psychopathe, il ira vérifier, il découvrira que thread.wait n'existe pas et il corrigera pour un truc genre join. J'avais utilisé wait parce que c'était un mot plus commun, immédiatement parlant, mais tu as raison de corriger.
J'aime beaucoup ta reformulation de la traduction des liens.

Tu as mon feu vert absolu 🟢👌

@awecx
Copy link
Author

awecx commented May 22, 2020

Je suis déçu de savoir que tu penses :

  • que je suis un psychopathe ;
  • que j'ai besoin d'aller vérifier la doc pour savoir si on utilise « wait » ou « join » dans Threading Python (genre…).

Merci pour ta relecture cependant 👍

@awecx awecx dismissed christopheNan’s stale review May 22, 2020 20:02

Remarques bien prises en compte.

@awecx awecx merged commit bd285e0 into python:3.8 May 22, 2020
@deronnax
Copy link
Collaborator

tiens, tu as pu fusionner malgré le "changes requested" de Christophe ?

@deronnax
Copy link
Collaborator

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

@awecx
Copy link
Author

awecx commented May 22, 2020

tiens, tu as pu fusionner malgré le "changes requested" de Christophe ?

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.

@awecx awecx deleted the contributing_5 branch August 19, 2020 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants