Thanks to visit codestin.com
Credit goes to github.com

Skip to content

tutorial/controlflow.po #940

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 26 commits into from
Dec 1, 2019
Merged

tutorial/controlflow.po #940

merged 26 commits into from
Dec 1, 2019

Conversation

alkimya
Copy link
Contributor

@alkimya alkimya commented Nov 1, 2019

No description provided.

Copy link
Contributor

@vpoulailleau vpoulailleau left a comment

Choose a reason for hiding this comment

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

Dans l'ensemble, cela me semble bien. Voici quelques petites remarques.

Copy link
Contributor

@vpoulailleau vpoulailleau left a comment

Choose a reason for hiding this comment

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

Pour moi ça va, mais je laisse les autres regarder.

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. Quelques remarques :

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 5, 2019

#976 aussi mais tu n'avais pas réservé le fichier @alkimya !

@alkimya
Copy link
Contributor Author

alkimya commented Nov 5, 2019

Aïe mince, je l'avais pris au tout début et on l'avait noté sur le tableau pour que personne ne le prenne, je n'ai pas pensé à le réserver après avec une Issue.

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 6, 2019

@alkimya aucun problème ! Soit je merge la mienne en premier et tu fait un rebase, soit on merge la tienne d'abord et je ferais le rebase

@deronnax
Copy link
Collaborator

deronnax commented Nov 6, 2019 via email

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 6, 2019

@alkimya Il me manque les dernières suggestions avant que je puisse merge ta PR 😁

@alkimya
Copy link
Contributor Author

alkimya commented Nov 6, 2019

Est-ce que c'est OK maintenant? Pas trop habitué à GitHub à part des projets perso, mais merci bien à tous.

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 6, 2019

Ca a l'air bon, je vais la relire et je te dit

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. Je t'ai fais quelques remarques.
Vérifie aussi que tu as bien pris en compte les suggestions de @vpoulailleau.

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 15, 2019

Hello @alkimya ! tu as encore la CI qui fail ;)

@alkimya
Copy link
Contributor Author

alkimya commented Nov 15, 2019

@Seluj78 Oui, je sais. Je m'en occupe au plus tôt

@@ -682,81 +707,107 @@ msgid ""
"passed by keyword argument, place an ``*`` in the arguments list just before "
"the first *keyword-only* parameter."
msgstr ""
"Pour marquer les paramètres comme *nommés*, indiquant que les paramètres "
"doivent être passés avec argument mot-clé, placez un ``*`` dans la liste des "
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
"doivent être passés avec argument mot-clé, placez un ``*`` dans la liste des "
"doivent être passés avec l'argument comme mot-clé, placez un ``*`` dans la liste des "

@alkimya
Copy link
Contributor Author

alkimya commented Nov 17, 2019

Mince, moi qui pensait avoir bien séparé mon travail avec des branches, je viens d'envoyer library/datetime.po sur la PR.

@Seluj78 Seluj78 removed the NEED REVIEWER This PR needs a reviewer :) label Nov 17, 2019
@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 17, 2019

@alkimya j'attends que tu résoude ca avant de review ;)

@Seluj78 Seluj78 changed the title Controlflow tutorial.controlflow.po Nov 17, 2019
@Seluj78 Seluj78 changed the title tutorial.controlflow.po tutorial/controlflow.po Nov 17, 2019
@alkimya
Copy link
Contributor Author

alkimya commented Nov 17, 2019

??
J'ai fait un revert, désindexé library/datetime.po, commité les changements de tutorial/controlflow.po, pushé. Mais library/datetime.po est toujours dans "Files changed".
Je fais quoi?

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.

Quelques remarques.


#: ../Doc/library/datetime.rst:31
msgid "Third-party library with expanded time zone and parsing support."
msgstr ""
"Bibliothèque tierce avec fuseau horaire étendu et prise en charge de "
"l'analyse."
Copy link

Choose a reason for hiding this comment

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

Suggested change
"l'analyse."
"l'analyse textuelle."

?


#: ../Doc/library/datetime.rst:38
msgid "Date and time objects may be categorized as \"aware\" or \"naive.\""
msgstr ""
"Les objets *date* et *time* peuvent être classés comme \"avisés\" ou \"naïfs"
Copy link

Choose a reason for hiding this comment

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

Suggested change
"Les objets *date* et *time* peuvent être classés comme \"avisés\" ou \"naïfs"
"Les objets *date* et *time* peuvent être classés comme « avisés » ou « naïfs»".


#: ../Doc/library/datetime.rst:38
msgid "Date and time objects may be categorized as \"aware\" or \"naive.\""
msgstr ""
"Les objets *date* et *time* peuvent être classés comme \"avisés\" ou \"naïfs"
"\"."
Copy link

Choose a reason for hiding this comment

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

Suggested change
"\"."


#: ../Doc/library/datetime.rst:164
msgid "A :class:`.datetime` object *d* is aware if both of the following hold:"
msgstr ""
"Un objet :class:`.datetime` *d* est avisé si les deux conditions suivantes "
"vérifient:"
Copy link

Choose a reason for hiding this comment

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

Suggested change
"vérifient:"
"vérifient :"

@@ -689,12 +708,10 @@ msgid "Notes:"
msgstr "Notes :"

#: ../Doc/library/datetime.rst:348
#, fuzzy
msgid "This is exact but may overflow."
msgstr "Ceci est exact, mais peut provoquer un débordement."
Copy link

Choose a reason for hiding this comment

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

Suggested change
msgstr "Ceci est exact, mais peut provoquer un débordement."
msgstr "Ceci est exact mais peut provoquer un débordement."


#: ../Doc/library/datetime.rst:852
msgid ""
"This method is functionally equivalent to :meth:`now`, but without a ``tz`` "
"parameter."
msgstr ""
"Cette méthode est fonctionnellement équivalente à :meth:`now`, mais sans "
"paramètre ``tz``."
Copy link

Choose a reason for hiding this comment

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

Suggested change
"paramètre ``tz``."
"le paramètre ``tz``."

@@ -1633,12 +1685,18 @@ msgid ""
"arbitrary ISO 8601 strings - it is only intended as the inverse operation "
"of :meth:`datetime.isoformat`."
msgstr ""
"Ceci ne supporte pas l'analyse arbitraire des chaînes de caractères ISO 8601 "
Copy link

Choose a reason for hiding this comment

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

Suggested change
"Ceci ne supporte pas l'analyse arbitraire des chaînes de caractères ISO 8601 "
"Ceci ne prend pas en charge pas l'analyse arbitraire des chaînes de caractères ISO 8601 "

"Ceci ne supporte pas l'analyse arbitraire des chaînes de caractères ISO 8601 "
"- il est uniquement destiné à l'opération inverse de :meth:`datetime."
"isoformat`. Un analyseur ISO 8601 plus complet, ``dateutil.parser.isoparse`` "
"est disponible dans le package tiers `dateutil <https://dateutil.readthedocs."
Copy link

Choose a reason for hiding this comment

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

Suggested change
"est disponible dans le package tiers `dateutil <https://dateutil.readthedocs."
"est disponible dans le paquet tiers `dateutil <https://dateutil.readthedocs."

"- il est uniquement destiné à l'opération inverse de :meth:`datetime."
"isoformat`. Un analyseur ISO 8601 plus complet, ``dateutil.parser.isoparse`` "
"est disponible dans le package tiers `dateutil <https://dateutil.readthedocs."
"io/en/stable/parser.html#dateutil.parser.isoparse>`__. Ceci ne supporte pas "
Copy link

Choose a reason for hiding this comment

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

Suggested change
"io/en/stable/parser.html#dateutil.parser.isoparse>`__. Ceci ne supporte pas "
"io/en/stable/parser.html#dateutil.parser.isoparse>`__. Ceci ne prend pas en charge "

"isoformat`. Un analyseur ISO 8601 plus complet, ``dateutil.parser.isoparse`` "
"est disponible dans le package tiers `dateutil <https://dateutil.readthedocs."
"io/en/stable/parser.html#dateutil.parser.isoparse>`__. Ceci ne supporte pas "
"l'analyse arbitraire des chaînes ISO 8601 - il est uniquement destiné à "
Copy link

Choose a reason for hiding this comment

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

Suggested change
"l'analyse arbitraire des chaînes ISO 8601 - il est uniquement destiné à "
"l'analyse arbitraire des chaînes ISO 8601 il est uniquement destiné à "

@alkimya
Copy link
Contributor Author

alkimya commented Nov 26, 2019

@Seluj78 Je suis revenu sur l'ancienne version de library/datetime.po pour cette PR.
J'ouvrirais une nouvelle PR pour les traductions manquantes de library/datetime.po avec les suggestions de @awecx

@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 26, 2019

@alkimya aucun problème, go ahead !

Copy link
Contributor

@vpoulailleau vpoulailleau left a comment

Choose a reason for hiding this comment

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

Voici les remarques remontées par padpo -p 940

@@ -76,14 +75,18 @@ msgstr ""
"de fin (comme en C), l'instruction :keyword:`!for` en Python itère sur les "
"é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. "
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
"caractères...), dans l'ordre dans lequel ils apparaissent dans la séquence. "
"caractères), dans l'ordre dans lequel ils apparaissent dans la séquence. "


#: ../Doc/tutorial/controlflow.rst:69
msgid ""
"Code that modifies a collection while iterating over that same collection "
"can be tricky to get right. Instead, it is usually more straight-forward to "
"loop over a copy of the collection or to create a new collection::"
msgstr ""
"Le code qui modifie une collection tout en itérant sur cette même collection "
"peut être délicat à mettre en place. Au lieu de cela, il est généralement "
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
"peut être délicat à mettre en place. Au lieu de cela, il est généralement "
"peut être délicat à mettre en place. Au lieu de cela, il est généralement "

@@ -310,7 +315,7 @@ msgstr ""
"qui utilisent ces chaînes de documentation pour générer automatiquement une "
"documentation en ligne ou imprimée, ou pour permettre à l'utilisateur de "
"naviguer de façon interactive dans le code ; prenez-en l'habitude, c'est 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
"naviguer de façon interactive dans le code ; prenez-en l'habitude, c'est une "
"naviguer de façon interactive dans le code ; prenez-en l'habitude, c'est une "


#: ../Doc/tutorial/controlflow.rst:539
msgid "A function definition may look like:"
msgstr ""
msgstr "Voici à quoi ressemble une définition de fonction :"
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 "Voici à quoi ressemble une définition de fonction :"
msgstr "Voici à quoi ressemble une définition de fonction :"


#: ../Doc/tutorial/controlflow.rst:614
msgid ""
"The second function ``pos_only_arg`` is restricted to only use positional "
"parameters as there is a ``/`` in the function definition::"
msgstr ""
"La deuxième fonction ``pos_only_arg`` restreint le passage des arguments par "
"position car il y a un ``/`` dans la définition de fonction ::"
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
"position car il y a un ``/`` dans la définition de fonction ::"
"position, car il y a un ``/`` dans la définition de fonction ::"


#: ../Doc/tutorial/controlflow.rst:625
msgid ""
"The third function ``kwd_only_args`` only allows keyword arguments as "
"indicated by a ``*`` in the function definition::"
msgstr ""
"La troisième fonction ``kwd_only_args`` n'autorise que les arguments nommés "
"comme l'indique le ``*`` dans la définition de fonction ::"
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 l'indique le ``*`` dans la définition de fonction ::"
"comme l'indique le ``*`` dans la définition de fonction ::"


#: ../Doc/tutorial/controlflow.rst:656
msgid ""
"Finally, consider this function definition which has a potential collision "
"between the positional argument ``name`` and ``**kwds`` which has ``name`` "
"as a key::"
msgstr ""
"Enfin, considérons cette définition de fonction qui a une collision "
"potentielle entre l'argument positionnel ``name`` et ``**kwds`` qui a "
"``name`` comme mot-clé ::"
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
"``name`` comme mot-clé ::"
"``name`` comme mot-clé ::"

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 PR

@JulienPalard JulienPalard merged commit 4503251 into python:3.8 Dec 1, 2019
@JulienPalard
Copy link
Member

@alkimya Pour créer une branche bien séparée, sans encombres et sans risques, une bon chemin est :

git fetch upstream
git checkout -b nouvelle_branche upstream/3.8

Le git fetch upstream télécharge les dernières modifs d'upstream (sans toucher a tes fichiers, donc tu peux le faire en était sur n'importe quelle branche, n'import equand)

Le git checkout -b nouvelle_branche upstream/3.8 crée la nouvelle branche nouvelle_branche basée sur upstream/3.8. upstream/3.8 c'est une référence vers "la tête de la branche 3.8 sur upstream" et pas "l'état de ta 3.8 locale".

Si bien sûr upstream est bien le nom de ton remote qui pointe vers https://github.com/python/python-docs-fr, ce que tu peux vérifier via git remote -v

@alkimya
Copy link
Contributor Author

alkimya commented Dec 1, 2019

Cool! Merci à tous

@alkimya alkimya deleted the controlflow branch December 1, 2019 19:29
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.

7 participants