Skip to content

Conversation

@alkimya
Copy link
Contributor

@alkimya alkimya commented Oct 31, 2019

No description provided.

@alkimya alkimya closed this Oct 31, 2019
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.

Merci pour cette très bonne contribution. Quelques remarques et interrogations.

#: ../Doc/faq/design.rst:152
msgid "Starting in Python 3.8, you can!"
msgstr ""
msgstr "A partir de Python 3.8, vous pouvez !"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
msgstr "A partir de Python 3.8, vous pouvez !"
msgstr "À partir de Python 3.8, vous pouvez !"
"This can be abstracted out into a wrapper function that can take a list and "
"tuples of field and order to sort them on multiple passes."
msgstr ""
"Ceci peut être résumé dans une fonction d'emballage qui peut prendre une "
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"Ceci peut être résumé dans une fonction d'emballage qui peut prendre une "
"Ce code peut être rendu plus générique en le plaçant dans une fonction qui encapsule le tout et prenant comme arguments une "

Ta version ressemble à de la traduction automatique.

"tuples of field and order to sort them on multiple passes."
msgstr ""
"Ceci peut être résumé dans une fonction d'emballage qui peut prendre une "
"liste et des tuples de champs et les trier sur plusieurs passes."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"liste et des tuples de champs et les trier sur plusieurs passes."
"liste et des couples (champ, sens) pour les ordonner sur plusieurs passes."
"éléments d'une séquence (qui peut être une liste, une chaîne de "
"caractères...), dans l'ordre dans lequel ils apparaissent dans la séquence. "
"Par exemple ::"
"Par exemple :"
Copy link
Contributor

Choose a reason for hiding this comment

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

Je ne sais pas : ce qui suit (dans le rendu) est du code, donc on devrait avoir le double deux-points. Cependant, il est vrai que la version originale n'en possède pas.

"There is no possible call that will make it return ``True`` as the keyword "
"``'name'`` will always to bind to the first parameter. For example::"
msgstr ""
"Il n'y a pas d'appel possible qui renverra ``True`` car le mot-clé ``'nom'`` "
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"Il n'y a pas d'appel possible qui renverra ``True`` car le mot-clé ``'nom'`` "
"Il n'y a pas d'appel possible qui renvoie ``True`` car le mot-clé ``'name'`` "
"arguments::"
msgstr ""
"Mais en utilisant ``/`` (arguments positionnels seulement), c'est possible "
"puisqu'il permet d'utiliser ``nom`` comme argument positionnel et ``'nom'`` "
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"puisqu'il permet d'utiliser ``nom`` comme argument positionnel et ``'nom'`` "
"puisqu'il permet d'utiliser ``name`` comme argument positionnel et ``'name'`` "
msgstr ""
"Mais en utilisant ``/`` (arguments positionnels seulement), c'est possible "
"puisqu'il permet d'utiliser ``nom`` comme argument positionnel et ``'nom'`` "
"comme clé dans les arguments nommés ::"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"comme clé dans les arguments nommés ::"
"comme mot-clé dans les arguments nommés ::"
"The use case will determine which parameters to use in the function "
"definition::"
msgstr ""
"Le cas d'utilisation déterminera les paramètres à utiliser dans la "
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"Le cas d'utilisation déterminera les paramètres à utiliser dans la "
"Le cas d'utilisation détermine les paramètres à utiliser dans la "
"parameter's name is modified in the future."
msgstr ""
"Dans le cas d'une API, n'utilisez que les paramètres positionnels pour "
"éviter d'interrompre les modifications de l'API si le nom du paramètre est "
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"éviter d'interrompre les modifications de l'API si le nom du paramètre est "
"éviter de casser l'API si le nom du paramètre est "
@Seluj78
Copy link
Collaborator

Seluj78 commented Oct 31, 2019

@alkimya pourquoi tu as close la PR ?

@alkimya
Copy link
Contributor Author

alkimya commented Oct 31, 2019

Matthieu m'a dit que c'était mieux de créer une branche dédiée et de fermer la PR

@Seluj78
Copy link
Collaborator

Seluj78 commented Oct 31, 2019

Yes ca marche, il a raison

@alkimya alkimya mentioned this pull request Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants