Skip to content

Conversation

@logistic-bot
Copy link
Contributor

No description provided.

Copy link

@awecx awecx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour cette bonne PR.
Je fini de lire demain.

logistic-bot and others added 23 commits November 1, 2019 12:23
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
…ordination

Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@awecx
Copy link

awecx commented Nov 5, 2019

Je crois avoir repondue a toutes vos attentes

Merci. Cependant les corrections que tu as acceptées ont cassé l'indentation (les 80 caractères max par ligne). Il faut que tu récupères ces modifications sur ton repo local (git pull), que tu les powrap, et que tu pousses le fichier ainsi indenté.

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 15, 2019

Hello @logistic-bot ! Petit ping ! 🏓

@logistic-bot
Copy link
Contributor Author

Hello @logistic-bot ! Petit ping !

Desole pour le manque d'acctivite, je suis tombe malade recament et ne suis pas tout a fait remis.
Je ne pourais pas travailer pendent un certain temps.

@deronnax
Copy link
Collaborator

Hello logistic-bot. J'espère que ça va mieux. Encore merci pour ce travail. Est-ce que tu peux cocher la case *allow edits from maintainers ? Je m'occupe de finir l'intégration de ta PR (ou alors si tu préfères le faire toi-même, il faut intégrer les dernières suggestions, faire le wrapping et merger 3.8 dans ta branche).

@logistic-bot
Copy link
Contributor Author

Sauf si je me trompe, la case est deja coché...

deronnax and others added 3 commits March 29, 2020 11:23
Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
@deronnax
Copy link
Collaborator

En effet, bien vu. Je m'en occupe.

@deronnax
Copy link
Collaborator

@awecx @christopheNan approval plz

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.

Quelques remarques.

Co-Authored-By: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
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.

beaucoup d'erreurs attrapées, bravo @christopheNan . J'ai juste une ou deux questions.

deronnax
deronnax previously approved these changes Mar 29, 2020
@deronnax deronnax dismissed stale reviews from themself via ff3a6fa March 29, 2020 15:26
Copy link

@awecx awecx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour cette PR.

"meth:`randrange` to handle arbitrarily large ranges."
msgstr ""
"Renvoie un entier Python avec *k* bits aléatoires. Cette méthode est fournie "
"avec le générateur MersenneTwister. D'autres générateurs peuvent également "
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"avec le générateur MersenneTwister. D'autres générateurs peuvent également "
"avec le générateur Mersenne-Twister. D'autres générateurs peuvent également "

À corriger upstream aussi.

Copy link
Collaborator

@deronnax deronnax May 2, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alors selon wikipedia (mon dieu à moi), c'est séparé par un espace, sans tirets. Je vais faire ainsi du coup.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

et c'est remonté upstream

Copy link

@awecx awecx May 2, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D'accord.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

et c'est corrigé \o/

"générateurs de nombres aléatoires. Cela implique que la plupart des "
"permutations d'une longue séquence ne peuvent jamais être générées. Par "
"exemple, une séquence de longueur 2080 est la plus grande qui puisse tenir "
"dans la période du générateur de nombres aléatoires Mersenne Twister."
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"dans la période du générateur de nombres aléatoires Mersenne Twister."
"dans la période du générateur de nombres aléatoires Mersenne-Twister."

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

du coup, non 🙃

deronnax and others added 3 commits May 2, 2020 18:26
Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
@deronnax deronnax requested a review from awecx May 2, 2020 20:45
@deronnax
Copy link
Collaborator

deronnax commented May 2, 2020

ah punaise avec le merge 3.8, la PR devient totalement vide. Je pense qu'il avait embarqué ses changements dans ses autres PR :/ .
Je ferme

@deronnax
Copy link
Collaborator

deronnax commented May 2, 2020

les changements sur random.po sont intégrés via la #916 et la fuzzy d'interpreter.po a été corrigée par Christophe il y a 3 jours via la #1230. Il ne reste plus rien à prendre dans cette PR, je la ferme.

@deronnax deronnax closed this May 2, 2020
@christopheNan
Copy link
Contributor

😞

@deronnax
Copy link
Collaborator

deronnax commented May 3, 2020

ah bah avec les PR qui trainent pendant des mois, y'a souvent du travail qui part à la poubelle (beaucoup).
Je vous ferai un petit mail de débrief sur mon travail de fossoyeur de PR de ces dernières semaines.

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.

5 participants