Skip to content

Traducido archivo library/sndhdr #1142

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 13 commits into from
Nov 21, 2020

Conversation

enrique-zarate-itti
Copy link
Contributor

@enrique-zarate-itti enrique-zarate-itti commented Nov 11, 2020

Closes #811

Copy link
Contributor

@mondeja mondeja 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 @enrique-zarate 🙏 Te he dejado unas sugerencias a ver qué te parecen.

enrique-zarate-itti and others added 4 commits November 12, 2020 11:47
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
@enrique-zarate-itti
Copy link
Contributor Author

Gracias por las correcciones @mondeja .
He aceptado las sugerencias. Me avisás si debo realizar algún paso extra por favor 😁.

Copy link
Contributor

@mondeja mondeja left a comment

Choose a reason for hiding this comment

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

Te dejo algunas correcciones más que se me pasaron. A parte de estas, hay un error que está marcando TravisCI. Indica que el ancho de alguno o varios de los msgids es demasiado. Para solucionarlo, debes corregirlo en el archivo library/sndhdr.po. La solución más fácil dependerá si estás en Linux, Windows o MacOS:

  • Linux y (quizás) MacOS: Instala las dependencias como se describe en la guía de contribución (pasos 5 y 6) y ejecuta powrap library/sndhdr.po.
  • Windows MacOS o Linux: Instala poedit, abre el archivo con el programa y guárdalo.

enrique-zarate-itti and others added 3 commits November 12, 2020 19:44
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
Co-authored-by: Álvaro Mondéjar <mondejar1994@gmail.com>
@mondeja
Copy link
Contributor

mondeja commented Nov 12, 2020

Sólo te falta arreglar el ancho de los msgids del archivo, con esto por mí estaría listo 👍

Por cierto, puedes hacer todos los cambios en un commit, no hace falta que realices un commit por sugerencia.

@enrique-zarate-itti
Copy link
Contributor Author

Sólo te falta arreglar el ancho de los msgids del archivo, con esto por mí estaría listo 👍

Por cierto, puedes hacer todos los cambios en un commit, no hace falta que realices un commit por sugerencia.

Disculpas 😅. Es mi segundo Pull request jaja.
He bajado los cambios, actualizado mi poedit (estoy en Windows), abierto y vuelto a guardar el archivo.
Espero que ahora funcione.

@mondeja
Copy link
Contributor

mondeja commented Nov 12, 2020

Ha vuelto a fallar. Comprueba que en tu configuración de Poedit tienes establecido los finales de renglón estilo Unix y el ajuste de línea a 79. Puedes encontrarlo en Editar -> Preferencias -> Avanzadas:

poedit-config

@enrique-zarate-itti
Copy link
Contributor Author

Tengo esa misma configuración:
imagen

Adicionalmente, al abrir el archivo me aparece este mensaje:
imagen

y al dar Fix language, me abre esta ventana:
imagen

Quizás deba cambiar algo aquí?

@mondeja
Copy link
Contributor

mondeja commented Nov 13, 2020

Prueba bajando el valor de "Wrap at" a 70.

@enrique-zarate-itti
Copy link
Contributor Author

imagen
Listo.

@@ -9,7 +9,7 @@ msgstr ""
"Project-Id-Version: Python 3.8\n"
"Report-Msgid-Bugs-To: \n"
"POT-Creation-Date: 2019-05-06 11:59-0400\n"
"PO-Revision-Date: 2020-11-12 20:42-0300\n"
"PO-Revision-Date: 2020-11-13 00:07-0300\n"
Copy link
Contributor

Choose a reason for hiding this comment

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

Este ha sido el único cambio. Quizás debas desmarcar la opción de "presevar el formato de los archivos existentes".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

He desmarcado la opción, guardado el archivo y vuelto a commitear.

@enrique-zarate-itti
Copy link
Contributor Author

Probaré dejando el Wrap como estaba originalmente.

@mondeja
Copy link
Contributor

mondeja commented Nov 14, 2020

Vale, ahora sí está formateando. Creía que powrap comprobaba un exceso de ancho, pero comprueba un ancho exacto, con lo que marcando a 70 tampoco pasa la construcción. Siento haberte dicho que lo marcases a 70 🙏 y perdona por esta montaña rusa.
Si no me equivoco entiendo que la solución es dejar desmarcado 'Preserve formatting of existing files' y definir 'Wrap at:' a 79.

@enrique-zarate-itti
Copy link
Contributor Author

imagen
Listo, así queda la configuración.
Y no te preocupes, para mi es un proceso de aprendizaje 😁.

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.

par de detallitos y mergeamos @enrique-zarate
Si te interesa saber por qué Retornar y no Devolver, tenemos un documento con varias cosas que hemos ido definiendo con el tiempo: https://python-docs-es.readthedocs.io/es/3.8/translation-memory.html sé que es dificil memorizarlas, con lo que no te preocupes no haberlo sabido de antes ;)

Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@enrique-zarate-itti
Copy link
Contributor Author

Buenas. Una consulta. En GitHub me aparece que el merging está bloqueado pero no sé si se deba a que me faltó hacer algún cambio o commit. Es esto cierto?

@cmaureir
Copy link
Collaborator

Buenas. Una consulta. En GitHub me aparece que el merging está bloqueado pero no sé si se deba a que me faltó hacer algún cambio o commit. Es esto cierto?

solo falta que alguien con permisos de merge lo apruebe, pero está todo OK y es normal. Mientas pasen los 2 checks ;)

@cmaureir cmaureir merged commit f95e7fb into python:3.8 Nov 21, 2020
@cmaureir
Copy link
Collaborator

Muchas gracias @enrique-zarate 🎉 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translate 'library/sndhdr.po'
3 participants