Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
awecx merged 20 commits intopython:3.8fromawecx:contributing_5
May 22, 2020
Merged

Conversation

awecx
Copy link

@awecxawecx commentedMay 17, 2020
edited
Loading

Les changements sont visiblesici.

Copy link
Contributor

@christopheNanchristopheNan 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).

@awecxawecx requested a review fromchristopheNanMay 18, 2020 19:46
@awecx
Copy link
Author

awecx commentedMay 18, 2020
edited
Loading

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.

@awecxawecx requested a review fromderonnaxMay 18, 2020 19:48
@deronnax
Copy link
Collaborator

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

@deronnaxderonnax self-assigned thisMay 18, 2020
Copy link
Collaborator

@deronnaxderonnax 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
Copy link
Author

@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).

@awecxawecx requested a review fromderonnaxMay 22, 2020 19:42
@deronnax
Copy link
Collaborator

Arf. En écrivantthread.join, je me suis dit "si c'est un psychopathe, il ira vérifier, il découvrira quethread.wait n'existe pas et il corrigera pour un truc genrejoin. 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

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 👍

deronnax reacted with heart emoji

@awecxawecx dismissedchristopheNan’sstale reviewMay 22, 2020 20:02

Remarques bien prises en compte.

@awecxawecx merged commitbd285e0 intopython:3.8May 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

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.

@awecxawecx deleted the contributing_5 branchAugust 19, 2020 20:40
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@deronnaxderonnaxderonnax approved these changes

@christopheNanchristopheNanAwaiting requested review from christopheNan

Assignees

@deronnaxderonnax

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@awecx@deronnax@christopheNan

[8]ページ先頭

©2009-2025 Movatter.jp