Skip to content

lib/doctest.po : de zéro à 72% #1663

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 21 commits into from
Sep 16, 2021
Merged

lib/doctest.po : de zéro à 72% #1663

merged 21 commits into from
Sep 16, 2021

Conversation

eviau
Copy link

@eviau eviau commented Jun 13, 2021

closes #1631 🎉

Je garde l'ambition de compléter cette traduction ! Un jour !

Je suis loin d'être convaincue de chacune de mes suggestions... je lirai attentivement les commentaires. 🕵️

JulienPalard
JulienPalard requested changes Jun 21, 2021
View reviewed changes
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.

Jolie trad' !

Relu jusqu'a la ligne 642.

JimmyLamothe
JimmyLamothe reviewed Jun 21, 2021
View reviewed changes
Copy link
Contributor

@JimmyLamothe JimmyLamothe left a comment

Choose a reason for hiding this comment

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

Correction de coquillles

@JulienPalard
Copy link
Member

JulienPalard commented Jun 22, 2021 •
edited
Loading

Attention j'ai rebasé sur la branche 3.10.
Ah bah non j'ai pas pu :

 ! [remote rejected]   library-doctest -> library-doctest (permission denied)
error: failed to push some refs to 'github.com:eviau/python-docs-fr.git'

Tu peux rebaser ta branche sur la 3.10, ou la laisser sur la 3.9 on pourra se charger de forward-porter après le merge sans aucun souci.

@eviau
Copy link
Author

eviau commented Jun 24, 2021

Il y a un commentaire ( celui-ci : #1663 (comment) ) sur lequel j'aimerais avoir un retour, sinon c'est bon - et je suis d'accord que docstring devrait être au féminin, car c'est la chaîne de documentation - du moins c'est ma logique.

@eviau eviau requested a review from JulienPalard July 30, 2021 20:30
@eviau
Copy link
Author

eviau commented Jul 30, 2021

Note: j'ai fait un "re-request review" mais c'était plus pour dire que j'étais prête à continuer, il n'y a pas de presse bien entendu ! :)

christopheNan
christopheNan previously requested changes Aug 5, 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.
J'ai revu le début. Je pense que la traduction est un peu trop littérale : n'hésite pas à te relire puis à reformuler pour rendre le texte plus naturel en français.

@eviau
Copy link
Author

eviau commented Aug 21, 2021

Merci pour les commentaires et la relecture, @christopheNan - en effet, le temps qui passe me fait réaliser quelques tournures de phrases qui pourraient être améliorées.

J'ai fait les changements suggérés, je reviens bientôt avec des corrections supplémentaires.

@eviau
Copy link
Author

eviau commented Sep 12, 2021

Relecture complétée !

Note: a priori les tests passaient mais en regardant le commit 00c94ba, j'ai vu qu'il y avait des erreurs soulevées, donc je les ai corrigées.

Suggestions, commentaires sont la bienvenue 😄

JulienPalard
JulienPalard approved these changes Sep 16, 2021
View reviewed changes
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.

Sacrée traduction, merci et bravo ! Ça devrait passer en prod en moins de 24.

@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

@JulienPalard JulienPalard merged commit cfd5147 into python:3.9 Sep 16, 2021
@eviau
Copy link
Author

eviau commented Sep 16, 2021

Excellent - ce fût un plaisir !

Merci pour les commentaires 😄

@JulienPalard JulienPalard mentioned this pull request Oct 16, 2021
Closed
JulienPalard pushed a commit that referenced this pull request Oct 16, 2021
@eviau @JulienPalard
lib/doctest.po : de zéro à 72% (#1663)
7bc5f3b 
(cherry picked from commit cfd5147)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JimmyLamothe JimmyLamothe JimmyLamothe left review comments

@christopheNan christopheNan christopheNan left review comments

@JulienPalard JulienPalard JulienPalard approved these changes

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/doctest.po
5 participants
@eviau @JulienPalard @PyDocTeur @JimmyLamothe @christopheNan