Skip to content

Update wave.po#622

Merged
JulienPalard merged 20 commits into
python:3.7from
christophetrinh:library/wave-po
Jun 2, 2019
Merged

Update wave.po#622
JulienPalard merged 20 commits into
python:3.7from
christophetrinh:library/wave-po

Conversation

@christophetrinh

Copy link
Copy Markdown
Contributor

Close #613

@Seluj78 Seluj78 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Super le CLoses #613 ! Merci encore pour ta contribution 😄

Attention toute fois a ne pas casser les tags rst. Les traducteurs on tendance a remplacer les ` par des ' et supprimer des espaces et bouger des :

N'hésite pas a utiliser les suggestions de github et d'utiliser le mode batch pour ne pas les accepter une par une

Avis aux autre reviewers: Relu actuellement que pour la syntaxe RST

Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Christophe Trinh added 2 commits March 23, 2019 17:48
@christophetrinh

Copy link
Copy Markdown
Contributor Author

Merci @Seluj78 et @christopheNan j'ai update :)

Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
@christopheNan

Copy link
Copy Markdown
Contributor

Merci @Seluj78 et @christopheNan j'ai update :)

La remarque pour « to support » est globale. Il en reste quelques uns.

@Seluj78

Seluj78 commented Apr 4, 2019

Copy link
Copy Markdown
Collaborator

@christophetrinh ping ? :)

@christophetrinh

Copy link
Copy Markdown
Contributor Author

Yep en effet, j'ai replace :) sorry pour le délai, par contre ça passe toujours pas côté ci :/

@JulienPalard

JulienPalard commented Apr 7, 2019

Copy link
Copy Markdown
Member

Bonjour @christophetrinh

Plusieurs petits soucis :

  • Les remarques dans la conversation au dessus sont parfois réparées mais pas marquées résolus, ça rend la conversation illisible, pourrais-tu marquer comme résulues celles qui le sont vraiment ?

  • Pourrais-tu configurer ton éditeur de po pour limiter la longueur des lignes à 79 caractères ? Ou utiliser https://pypi.org/project/powrap/ si ton éditeur ne te le permet pas. C'est normalement la configuration par défaut de poedit aussi. Le souci est que dans ta PR actuelle, tu change la positition des retours a la ligne, meme dans les messages en anglais, rendant le diff illisible.

  • Tu as quelques erreurs d'orthographe remontés par pospell dans travis (tu peux cliquer sur "detail" pour les voir) :

library/wave.po:77:wave
library/wave.po:236:mod
library/wave.po:281:nchannels
library/wave.po:281:sampwidth
library/wave.po:281:framerate
library/wave.po:281:nframes
library/wave.po:281:comptype
library/wave.po:281:compname

Je pense que le souci de la ligne 281 est une espace en trop avant le double backtick fermant, et c'est cette erreur qui cause aussi l'erreur de Sphinx plus bas (la syntaxe rest n'accepte pas cet espace avant le double backtick).

@christophetrinh

Copy link
Copy Markdown
Contributor Author

Hello et merci pour le retour !

J'ai résolu les remarques et powrap le fichier.

Il reste un dernier fail de pospell que j'arrive pas à trouver:

library/wave.po:80:wave

Comment thread library/wave.po Outdated
Co-Authored-By: christophetrinh <christophe.trinh.94@gmail.com>
@christophetrinh

Copy link
Copy Markdown
Contributor Author

C'est tout bon pour Travis merci à vous guys :)

@Seluj78 Seluj78 requested a review from christopheNan April 13, 2019 12:48
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
@Seluj78 Seluj78 requested a review from JulienPalard April 13, 2019 12:59
Co-Authored-By: christophetrinh <christophe.trinh.94@gmail.com>
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
@Seluj78 Seluj78 added the NEED REVIEWER This PR needs a reviewer :) label Apr 17, 2019
@Seluj78

Seluj78 commented Apr 18, 2019

Copy link
Copy Markdown
Collaborator

@christophetrinh ping 🚀

Co-Authored-By: christophetrinh <christophe.trinh.94@gmail.com>
@Seluj78

Seluj78 commented Apr 24, 2019

Copy link
Copy Markdown
Collaborator

@christophetrinh ping pour la CI ;)

@christophetrinh

Copy link
Copy Markdown
Contributor Author

@Seluj78 Merci pour le ping ça devrait être bon cette fois-ci :)

@deronnax deronnax left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

J'ai bossé un peu dans l'audio et j'ai pas vraiment de sources à vous filer mais je vous jure sur la tête de Bernadette Chirac que la traduction de audio frame en français c'est trame audio.
Je ferai une PR pour changer ça une fois cette PR mergée.

Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
christophetrinh and others added 2 commits May 14, 2019 09:46
Co-Authored-By: Mathieu Dupuy <deronnax@users.noreply.github.com>
@deronnax

deronnax commented May 15, 2019

Copy link
Copy Markdown
Collaborator

bon allez je t'embête : remplace toutes les occurences de "fenêtre audio" par "trame audio" et ta PR sera parfaite 👌

(EDIT: et je viens de trouver une parenthèse en trop)

Comment thread library/wave.po Outdated
christophetrinh and others added 2 commits May 16, 2019 19:41
Co-Authored-By: Mathieu Dupuy <deronnax@users.noreply.github.com>

@deronnax deronnax left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

j'approuve cette PR de toute beauté

@JulienPalard JulienPalard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

J'ai quelques petites remarques.

Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
Comment thread library/wave.po Outdated
@deronnax

Copy link
Copy Markdown
Collaborator

Hmm, le module wave faisait parti du premier brouillon de la PEP 594, qui listait les modules qui allaient être supprimés. Heureusement le module wave a été l'un des rares à avoir été finalement épargné, sinon cette trad partait à la poubelle dans deux versions de python :)

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

Seluj78 commented May 31, 2019

Copy link
Copy Markdown
Collaborator

@christophetrinh ping 😄

@christophetrinh

Copy link
Copy Markdown
Contributor Author

@Seluj78 c'est bon!

@Seluj78 Seluj78 added NEED REVIEWER This PR needs a reviewer :) and removed awaiting changes labels Jun 1, 2019
@JulienPalard JulienPalard merged commit 7f9aff1 into python:3.7 Jun 2, 2019
@JulienPalard

Copy link
Copy Markdown
Member

@christophetrinh C'est mergé ! Merci pour cette trad \o/

Ça prend maximum 3h pour arriver en production après le merge, si tu veux voir ton travail sur docs.python.org/fr/3/.

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 library/wave.po

5 participants