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

Skip to content

Fuzzies Translation - datetime #1352

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 6 commits into from
Jun 28, 2020
Merged

Fuzzies Translation - datetime #1352

merged 6 commits into from
Jun 28, 2020

Conversation

alkimya
Copy link
Contributor

@alkimya alkimya commented Jun 26, 2020

Closes #1343

Copy link
Member

@JulienPalard JulienPalard left a comment

Choose a reason for hiding this comment

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

Top merci pour cette PR !

@JulienPalard JulienPalard merged commit 7bc46ce into python:3.9 Jun 28, 2020
@deronnax
Copy link
Collaborator

@christopheNan je peux te demander une revue là-dessus ? Y'a 2-3 trucs qui me turlupinent (notamment le changement "dépasser les bornes" en "déborder", que je pense moins correct).

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.

Pour @deronnax : je comprends que « débordement » lève une OverflowError alors que « dépasser les bornes » n'est pas aussi explicite (cependant, je n'ai pas vérifié que les débordements levaient effectivement une exception).

Comment on lines +40 to +41
"essentiellement tournée vers l'efficacité pour extraire des attributs pour "
"les manipuler et les formater pour l'affichage."
Copy link
Contributor

Choose a reason for hiding this comment

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

3 pour à la suite dans la phrase ! Je propose :
« l'attention lors de l'implémentation s'est essentiellement portée sur l'efficacité de l'extraction des attributs en vue de leur manipulation et formatage pour l'affichage. »

Copy link
Contributor Author

Choose a reason for hiding this comment

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

j'avais mis dans mon premier commit "l'accent est mis sur l'extraction efficace des attributs pour le formatage et la manipulation des sorties." avant de remettre la traduction originale

msgid "Aware and Naive Objects"
msgstr "Pour un objet avisé :"
msgstr "Objets « Avisés » et « Naïfs »"
Copy link
Contributor

Choose a reason for hiding this comment

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

Pas de majuscule à chaque mot dans les titres en français.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, je laisse les guillemets?

"appliquer et des politiques d'ajustement de l'heure comme les informations "
"sur les fuseaux horaires et l'heure d'été pour se situer de façon relative "
"par rapport à d'autres objets avisés. Un objet avisé est utilisé pour "
"représenter un moment précis de l'histoire qui n'est pas ouvert à "
Copy link
Contributor

Choose a reason for hiding this comment

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

« un moment précis de l'histoire ». J'écrirais plus simplement « un instant précis » car je trouve « histoire » ambigu : ce n'est pas l'Histoire et on ne raconte pas une histoire.

@@ -208,7 +194,7 @@ msgid ""
msgstr ""
"Un temps idéalisé, indépendant d'une date particulière, en supposant qu'une "
"journée est composée d'exactement 24\\*60\\*60 secondes (il n'y a pas ici de "
"notion de \"seconde bissextile\"). Attributs : :attr:`hour`, :attr:"
"notion de \"seconde bissextile\"). Attributs : :attr:`hour`, :attr:"
Copy link
Contributor

Choose a reason for hiding this comment

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

leap second : seconde intercalaire.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

merci

"Tous les arguments sont requis. Les arguments peuvent être des entiers, "
"dans les intervalles suivant :"
"Tous les arguments sont requis. Les arguments peuvent être des entiers, dans "
"les intervalles suivant :"
Copy link
Contributor

Choose a reason for hiding this comment

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

suivants

"bornes gérées par la fonction C :c:func:`localtime` de la plateforme, et "
"une :exc:`OSError` en cas d'échec de :c:func:`localtime`. Il est commun "
"d'être restreint aux années entre 1970 et 2038. Notez que sur les systèmes "
"non *POSIX* qui incluent les secondes de décalage dans leur notion "
Copy link
Contributor

Choose a reason for hiding this comment

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

les secondes intercalaires.

@alkimya
Copy link
Contributor Author

alkimya commented Jul 1, 2020

Et comment je fais vu que c'est mergé? Je n'ai pas effacé la branche en locale, je continue à faire les changements et je push et rouvre une PR?

@deronnax
Copy link
Collaborator

deronnax commented Jul 1, 2020

ça devrait fonctionner ;)

@JulienPalard
Copy link
Member

Et comment je fais vu que c'est mergé? Je n'ai pas effacé la branche en locale, je continue à faire les changements et je push et rouvre une PR?

Tes commits, une fois mergés, n'ont pas le même sha1 (on les squash et on les rebase, on broie tout au merge des PR), donc ce n'est pas une bonne idée de continuer sur ta branche.

Il vaut mieux repartir à neuf :

git fetch upstream  # Télécharger les dernières modifications
git checkout -b fix-datetime upstream/3.9  # Créer une branche neuve a partir du dernier commit de la 3.9 d'upstream

@alkimya alkimya deleted the datetime branch July 2, 2020 01:18
deronnax pushed a commit that referenced this pull request Jul 3, 2020
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.

Je travaille sur library/datetime.po
4 participants