Skip to content

Conversation

@Seluj78
Copy link
Collaborator

@Seluj78 Seluj78 commented May 23, 2019

Fixes #735

@Seluj78 Seluj78 added the NEED REVIEWER This PR needs a reviewer :) label May 23, 2019
@Seluj78 Seluj78 self-assigned this May 23, 2019
@deronnax
Copy link
Collaborator

je viens de regarder la doc française de datetime, et ils n'utilisent pas passage pour désigner fold, ils gardent fold tel quel en italique. Je pense qu'il faut plutôt faire ça. Surtout que si tu me dis "passage" en parlant de datetime, là comme ça à froid je comprends absolulement pas qu'il s'agit du folding ;)

@Krilivye
Copy link
Contributor

J'aime bien la représentation du passage ut0-ut1 : https://www.python.org/dev/peps/pep-0495/#why-fold
Pour être cohérent il est peut être plus simple de mettre fold et de donner le lien vers la pep?

@Seluj78 Seluj78 changed the title Fixed fuzzies Fixed fuzzies in c-api/datetime.po May 24, 2019
Copy link
Contributor

@Krilivye Krilivye left a comment

Choose a reason for hiding this comment

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

Je garderais plutôt et le fold avec le liens vers la description ;) https://docs.python.org/fr/3.7/library/datetime.html#datetime.datetime.fold

@christopheNan
Copy link
Contributor

je viens de regarder la doc française de datetime, et ils n'utilisent pas passage pour désigner fold, ils gardent fold tel quel en italique.

Une traduction de fold est pli et il est dit dans la PEP 495 que Guido pense que fold a les bonnes connotations.
Je pense donc qu'il faut traduire par pli (éventuellement en précisant que c'est le paramètre fold) et remonter un ticket vers la doc française de datetime en proposant pli.

@JulienPalard
Copy link
Member

J'ai rien contre pli, même si j'ai une toute légère préférence pour passage je pense. En tout cas le laisser en anglais peut aider (je veux dire le traduire et entre parenthèse en italique mettre le nom du paramètre).

@JulienPalard JulienPalard added awaiting changes and removed NEED REVIEWER This PR needs a reviewer :) labels May 28, 2019
@Seluj78 Seluj78 added NEED REVIEWER This PR needs a reviewer :) and removed awaiting changes labels May 31, 2019
@JulienPalard
Copy link
Member

Je voit un "pli" et un "passage".

@JulienPalard JulienPalard added awaiting changes and removed NEED REVIEWER This PR needs a reviewer :) labels May 31, 2019
@Seluj78 Seluj78 added NEED REVIEWER This PR needs a reviewer :) and removed awaiting changes labels May 31, 2019
@Seluj78 Seluj78 merged commit 7580eaf into python:3.7 Jun 1, 2019
@Seluj78 Seluj78 deleted the c-api-datetime branch June 1, 2019 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NEED REVIEWER This PR needs a reviewer :)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Je travaille sur c-api/datetime.po

5 participants