Skip to content

Update wave.po #622

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 20 commits into from
Jun 2, 2019
Merged

Update wave.po #622

merged 20 commits into from
Jun 2, 2019

Conversation

christophetrinh
Copy link
Contributor

Close #613

Copy link
Collaborator

@Seluj78 Seluj78 left a comment

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

Christophe Trinh added 2 commits March 23, 2019 17:48
@christophetrinh
Copy link
Contributor Author

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

@christopheNan
Copy link
Contributor

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

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

@Seluj78
Copy link
Collaborator

Seluj78 commented Apr 4, 2019

@christophetrinh ping ? :)

@christophetrinh
Copy link
Contributor Author

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

@JulienPalard
Copy link
Member

JulienPalard commented Apr 7, 2019

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
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

Co-Authored-By: christophetrinh <christophe.trinh.94@gmail.com>
@christophetrinh
Copy link
Contributor Author

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

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

Seluj78 commented Apr 18, 2019

@christophetrinh ping 🚀

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

Seluj78 commented Apr 24, 2019

@christophetrinh ping pour la CI ;)

@christophetrinh
Copy link
Contributor Author

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

Copy link
Collaborator

@deronnax deronnax left a comment

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.

christophetrinh and others added 2 commits May 14, 2019 09:46
Co-Authored-By: Mathieu Dupuy <deronnax@users.noreply.github.com>
@deronnax
Copy link
Collaborator

deronnax commented May 15, 2019

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)

christophetrinh and others added 2 commits May 16, 2019 19:41
Co-Authored-By: Mathieu Dupuy <deronnax@users.noreply.github.com>
Copy link
Collaborator

@deronnax deronnax left a comment

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é

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.

J'ai quelques petites remarques.

@deronnax
Copy link
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
Copy link
Collaborator

Seluj78 commented May 31, 2019

@christophetrinh ping 😄

@christophetrinh
Copy link
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
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy