Skip to content

Traduction de library/configparser.po #1626

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 28, 2021
Merged

Traduction de library/configparser.po #1626

merged 1 commit into from
May 28, 2021

Conversation

dmerejkowsky
Copy link
Contributor

No description provided.

@dmerejkowsky
Copy link
Contributor Author

Bon j'en suis à peu près à 20% mais j'ai deux/trois questions si quelqu'un passe par là.

J'ai très envie d'utiliser "parseur" pour parler des instances ConfigParser - même si ça va à l'encontre des règles dans CONTRIBUTING. Je suis pas sûr qu'un lecteur non averti comprendra qu'on parle d'instances de classe si on utilise le mot "lecteur" ou "analyseur"

Le contenu anglais utilise à la fois fallback et default pour parler de deux concepts différents. Je suis pas certain de ma traduction en "de substitution" et "par défaut".

J'ai pour le moment aucune idée sur la façon de traduire "special default section" et "Mapping Access Protocol", mais je ne désespère pas ;)

christopheNan
christopheNan suggested changes May 7, 2021
View reviewed changes
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 contribution.
Peut-être utiliser « valeur de repli » pour « fallback » ?
Mettre INI entre astérisques permettrait de bien distinguer que ce n'est pas du français.

@dmerejkowsky dmerejkowsky force-pushed the library-configparser branch from e502049 to ae4bd36 Compare May 16, 2021 15:06
christopheNan
christopheNan suggested changes May 19, 2021
View reviewed changes
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 ce travail de très grande qualité. Quelques remarques liées à des conventions utilisées dans cette traduction.

christopheNan
christopheNan previously approved these changes May 27, 2021
View reviewed changes
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.

👍

@PyDocTeur
Copy link

ON Y EST PRESQUE ! Un p'tit label automerge et je merge ça !

Disclaimer

Je suis un robot fait par l'équipe de l'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 the Translation 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

@christopheNan christopheNan added the 🤖 automerge The PR will be auto merged label May 27, 2021
@PyDocTeur
Copy link

Ping @dmerejkowsky & @christopheNan !
Il m'est actuellement impossible de merge cette pull request.

L'état actuel de merge la PR est draft.

Merci de rebase ou de résoudre les conflits !

Disclaimer

Je suis un robot fait par l'équipe de l'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 the Translation 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: automerge_approved_testok)
PyDocTeur v1.12.0

@dmerejkowsky dmerejkowsky marked this pull request as ready for review May 27, 2021 15:52
@dmerejkowsky
Copy link
Contributor Author

Oops, forgot to switch from "draft" to " ready for review"

Hoping @PyDocTeur will merge it now

@dmerejkowsky dmerejkowsky force-pushed the library-configparser branch from bc4a821 to ff627ff Compare May 27, 2021 15:57
@PyDocTeur
Copy link

Y'a-il un humain dans le coin pour relire ? CC @christopheNan

Disclaimer

Je suis un robot fait par l'équipe de l'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 the Translation 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: automerge)
PyDocTeur v1.12.0

@dmerejkowsky
Copy link
Contributor Author

dmerejkowsky commented May 27, 2021 •
edited
Loading

J'ai rebasé et utilisé push -f

Note pour les hunains: j'ai vérifié que configparser.po n'avait pas changé :

$ git diff bc4a821..ff627ff
diff --git a/library/configparser.po b/library/configparser.po
index d75fa020..4dbb36ae 100644
--- a/library/configparser.po
+++ b/library/configparser.po
@@ -5,7 +5,7 @@ msgid ""
 
 Project-Id-Version: Python 3\n
 Report-Msgid-Bugs-To: \n
-POT-Creation-Date: 2020-10-01 16:00+0200\n
+POT-Creation-Date: 2021-05-19 22:36+0200\n
 Language-Team: FRENCH <traductions@lists.afpy.org>\n
 Language: fr\n
 MIME-Version: 1.0\n

@dmerejkowsky dmerejkowsky force-pushed the library-configparser branch from ff627ff to 06cb1ab Compare May 27, 2021 16:10
@dmerejkowsky
Copy link
Contributor Author

Et pis tant qu'à faire j'ai tout compressé dans un seul commit, si jamais y a un autre humain qui veut relire ;)

@JulienPalard JulienPalard merged commit b26d68a into python:3.9 May 28, 2021
@JulienPalard
Copy link
Member

Merci Dimitri!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JulienPalard JulienPalard JulienPalard approved these changes

@christopheNan christopheNan christopheNan left review comments

Assignees
No one assigned
Labels
🤖 automerge The PR will be auto merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dmerejkowsky @PyDocTeur @JulienPalard @christopheNan