Skip to content

Traducción de archivo library/windsound.po #791 #942

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 6 commits into from
Oct 5, 2020
Merged

Traducción de archivo library/windsound.po #791 #942

merged 6 commits into from
Oct 5, 2020

Conversation

ayosefiguera
Copy link
Contributor

@ayosefiguera ayosefiguera commented Oct 4, 2020 •
edited by cmaureir
Loading

Closes #791

cmaureir
cmaureir requested changes Oct 4, 2020
View reviewed changes
Copy link
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

Gracias por la traducción @ayosefiguera 🎉 Unos cambios menores a la traducción. Ahora pasará el build si los aceptas. Recuerda que puedes agregarlos todos juntos usando la interfaz de Github. Si no lo haces así, y lo cambias localmente, recuerda marcar cada sugerencia con el botón "Resolve conversation" para mirar cuales aceptaste.

@ayosefiguera
Copy link
Contributor Author

Gracias, me apunto las correcciones para futuras aportaciones.

@cmaureir
Copy link
Collaborator

cmaureir commented Oct 4, 2020

Travis se quedó pegado, voy a cerrar/abrir el PR para otro check.

@cmaureir cmaureir closed this Oct 4, 2020
@cmaureir cmaureir reopened this Oct 4, 2020
cmaureir
cmaureir reviewed Oct 4, 2020
View reviewed changes
Letram
Letram reviewed Oct 5, 2020
View reviewed changes
Copy link
Contributor

@Letram Letram left a comment

Choose a reason for hiding this comment

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

Te arreglo un par de comillas que están puestas con otro estilo!

@ayosefiguera
Copy link
Contributor Author

Hola es necesario algún otro cambio ?

@cmaureir
Copy link
Collaborator

cmaureir commented Oct 5, 2020

Hola es necesario algún otro cambio ?

Sí, siempre puedes hacer click en 'Details' en la sección de Travis y luego en 'The build' acá un link directo: https://travis-ci.com/github/python/python-docs-es/builds/188071389
Eso te dice:

$ pospell -p dict.txt -l es_AR -l es_ES **/*.po
library/winsound.po:98:unnombre
The command "pospell -p dict.txt -l es_AR -l es_ES **/*.po" exited with 255.

Es decir que en tu código está escrito junto eso "unnombre", te dejo ahora la correción

@cmaureir
Copy link
Collaborator

cmaureir commented Oct 5, 2020

Veo que le has dado a 'Resolve conversation' @ayosefiguera pero no has aplicado los cambios... Por favor, revisa esto: https://www.youtube.com/watch?v=SH8HGBPASYY

ayosefiguera and others added 3 commits October 5, 2020 18:57
@ayosefiguera @cmaureir @Letram
Apply suggestions from code review
1785026 
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Carlos Martel Lamas <Letram@users.noreply.github.com>
@ayosefiguera @cmaureir
Apply suggestions from code review
c728b55 
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@ayosefiguera
Update winsound.po
0c04e9d
@cmaureir
Copy link
Collaborator

cmaureir commented Oct 5, 2020

Falta powrap, si tienes problemas estando en Windows, avisame y lo paso yo ;)

@ayosefiguera
Copy link
Contributor Author

Hola @cmaureir al intentar pasar el powrap me genera errores.
Podrías pasarlo tú.
Un saludo

@ayosefiguera
Copy link
Contributor Author

Gracias

@cmaureir cmaureir merged commit 06b990c into python:3.8 Oct 5, 2020
@cmaureir
Copy link
Collaborator

cmaureir commented Oct 5, 2020

Muchas gracias @ayosefiguera 🎉 🎉

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

@Letram Letram Letram left review comments

@cmaureir cmaureir cmaureir 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.

Translate 'library/winsound.po'
3 participants
@ayosefiguera @cmaureir @Letram