Skip to content

Conversation

@Dygouin
Copy link
Contributor

@Dygouin Dygouin commented Jan 11, 2019

No description provided.

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 propositions

christopheNan and others added 16 commits January 12, 2019 00:15
Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Update with suggestions

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
Merci pour le code review

Co-Authored-By: DeltaAlphaIotaLamdaAlphaNu <gouindylan7@gmail.com>
@Dygouin
Copy link
Contributor Author

Dygouin commented Jan 12, 2019

yay les checks sont passés

library/errno.po Outdated
#: ../Doc/library/errno.rst:123
msgid "No such device"
msgstr ""
msgstr "Périphérique inexistant"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Est ce que il y aurait pas un autre mot pour device? Car je suppose que cette erreur fait référence à /dev/ et on y trouve des devices physiques comme virtuels et a mon sens périphérique fait trop référence a un clavier par exemple

Copy link
Contributor Author

Choose a reason for hiding this comment

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

système, appareil, (dispositif) je préfere le 3ème

Copy link
Contributor

Choose a reason for hiding this comment

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

"Dispositif" me semble bien.

library/errno.po Outdated
#: ../Doc/library/errno.rst:168
msgid "No space left on device"
msgstr ""
msgstr "Plus de place sur le périphérique"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pareil ici, périphérique me semble etrange utilié ici puisqu'on fait référence a un disque (peut importe son type) ou alors on fait référence a la machine en elle meme (typiquement sur macos, lorsque un home est full (a 42 par exemple) on a cette erreur.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dispositif est peut être mieux?

@Dygouin
Copy link
Contributor Author

Dygouin commented Jan 12, 2019

Du coup j'attend tes réponses avant de repush

@Seluj78
Copy link
Collaborator

Seluj78 commented Jan 12, 2019

@DeltaAlphaIotaLamdaAlphaNu responses a ? :)

@Dygouin
Copy link
Contributor Author

Dygouin commented Jan 12, 2019

Les requests que j'ai laissée ouvertes

@Seluj78
Copy link
Collaborator

Seluj78 commented Jan 14, 2019

@DeltaAlphaIotaLamdaAlphaNu tu as tes reponses 👍

@Dygouin
Copy link
Contributor Author

Dygouin commented Jan 14, 2019

@Seluj78 dès que je suis à la maison je te fais le dernier push pour que tu puisses me review (je suis a 42 et je galère pour push d'ici).

@Dygouin
Copy link
Contributor Author

Dygouin commented Jan 14, 2019

Ça devrait être bon là

Copy link
Collaborator

@Seluj78 Seluj78 left a comment

Choose a reason for hiding this comment

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

La CI passe 👌

@christopheNan christopheNan merged commit dfff534 into python:3.7 Jan 15, 2019
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