Skip to content

using/windows.po #1886

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 8 commits into from
May 26, 2022
Merged

using/windows.po #1886

merged 8 commits into from
May 26, 2022

Conversation

deronnax
Copy link
Collaborator

@deronnax deronnax commented May 24, 2022

il me reste ~10 entrées à traduire

@deronnax deronnax marked this pull request as draft May 24, 2022 11:23
@deronnax
Copy link
Collaborator Author

@christopheNan quel est à ton avis sur cette suggestion de padpo ?

warning: Found "support" that is not translated in "prendre en charge", "prend en charge", "prennent en charge", "implémenter", "implémente" or "implémentent" in ###Gestion des chemins longs.###.�[0m

@vpoulailleau
Copy link
Contributor

@christopheNan quel est à ton avis sur cette suggestion de padpo ?

Cette suggestion provient du glossaire dans le guide de contribution (j'ai pas lu ta traduction, mais si la suggestion n'est pas bonne, ça peut valoir le coup de mettre à jour le guide).

@christopheNan
Copy link
Contributor

Je pense que l'on peut ignorer la suggestion. La traduction de to support peut être variée, la vérification de padpo étant là surtout pour éviter d'avoir supporter en français.

@jeanas
Copy link
Collaborator

jeanas commented May 25, 2022

Je suis d'accord avec Christophe. Je préfère voir le tableau de CONTRIBUTING.rst comme des propositions, la traduction peut dépendre du contexte et il peut être approprié de reformuler (par contre, « supporter » a un sens complètement différent, donc mieux vaut l'éviter).

@deronnax deronnax marked this pull request as ready for review May 26, 2022 16:38
@deronnax
Copy link
Collaborator Author

ready for review

@deronnax deronnax requested a review from christopheNan May 26, 2022 16:38
@deronnax
Copy link
Collaborator Author

deronnax commented May 26, 2022

à merger rapidement parce qu'au doigt mouillé, 80% de nos lecteurs en français sont sous Windows

Copy link
Contributor

@vpoulailleau vpoulailleau left a comment

Choose a reason for hiding this comment

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

Bon, faut pas tout prendre à la lettre, je ne suis pas sûr de tout.

deronnax and others added 3 commits May 26, 2022 20:29
@PyDocTeur
Copy link

Rajoutez-moi un automerge et ca part en prod !


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

@vpoulailleau
Copy link
Contributor

Oups, j'ai pas le commit bit, j'ai l'impression que PyDocTeur s'en satisfait, mais ça veut dire que n'importe qui de pas sérieux (comme moi 😉) peut faire croire à PyDocTeur que la doc est bien traduite (@JulienPalard @Seluj78), c'est bien le comportement voulu ?

@deronnax deronnax added the 🤖 automerge The PR will be auto merged label May 26, 2022
@PyDocTeur
Copy link

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

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

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

@deronnax
Copy link
Collaborator Author

il check le status avant de merger donc c'est bon

@vpoulailleau
Copy link
Contributor

il check le status avant de merger donc c'est bon

Il nous dit « Merci de rebase ou de résoudre les conflits ! », ce n'est pas ici le cas, on attend un reviewer qualifié ! Ça mériterait un petit complément dans le message de PyDocTeur. Mais bon, au moins, c'est pas mergé à la va vite.

@jeanas
Copy link
Collaborator

jeanas commented May 26, 2022

Oui c'est un bug connu de PyDocTeur. (D'ailleurs, PyDocTeur en l'état n'a pas tant d'intérêt que ça, il était destiné à appliquer powrap automatiquement mais ce n'est jamais arrivé…)

@jeanas jeanas merged commit 2c827d8 into python:3.10 May 26, 2022
JulienPalard pushed a commit that referenced this pull request Sep 13, 2022
Co-authored-by: Vincent Poulailleau <vpoulailleau@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 automerge The PR will be auto merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants