Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Traduction de library/zoneinfo.po#1879

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
christopheNan merged 5 commits intopython:3.10fromjeanas:zoneinfo
May 27, 2022
Merged

Traduction de library/zoneinfo.po#1879

christopheNan merged 5 commits intopython:3.10fromjeanas:zoneinfo
May 27, 2022

Conversation

jeanas
Copy link
Collaborator

Reprise de#1579

Closes#1532

Gaasmann
Gaasmann previously approved these changesMay 22, 2022
Copy link
Contributor

@GaasmannGaasmann left a comment

Choose a reason for hiding this comment

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

PR revue, rien à redire. J'approuve mais je ne suis pas sur d'avoir voix au chapitre en la matière :-)

@PyDocTeur
Copy link

Rajoutez-moi unautomerge et ca part en prod !

Disclaimer

Je suis un robot fait par l'équipe del'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by theTranslation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: approved)
PyDocTeur v1.12.0

@jeanas
Copy link
CollaboratorAuthor

Non, toutes les relectures sont utiles ! La tienne ne débloque pas la fusion de la PR (seules les relectures de personnes avec les droits d'accès le font), mais cela n'empêche pas de faire des remarques pertinentes, et même si tu n'as rien à redire, le fait de manifester ton approbation permet d'avoir une idée du niveau de relecture qu'a reçu la PR.

Gaasmann reacted with thumbs up emoji

Copy link
Contributor

@christopheNanchristopheNan left a comment

Choose a reason for hiding this comment

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

Je mettrais au présent les descriptions qui ne font pas référence explicitement à un événement futur, comme préconisé dansCONTRIBUTING.

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Copy link
Contributor

@christopheNanchristopheNan left a comment

Choose a reason for hiding this comment

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

👍 Merci !

@christopheNanchristopheNan merged commit1e8e99b intopython:3.10May 27, 2022
@jeanasjeanas deleted the zoneinfo branchMay 27, 2022 08:51
JulienPalard pushed a commit that referenced this pull requestSep 13, 2022
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>Co-authored-by: Loc Cosnier <loc.cosnier@pm.me>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@christopheNanchristopheNanchristopheNan approved these changes

@GaasmannGaasmannGaasmann left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Je travaille sur library/zoneinfo.po
5 participants
@jeanas@PyDocTeur@Gaasmann@christopheNan@alkimya
[8]ページ先頭
©2009-2025 Movatter.jp