-
Notifications
You must be signed in to change notification settings - Fork 396
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
Conversation
There was a problem hiding this 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.
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>
Gracias por las correcciones @mondeja . |
There was a problem hiding this 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.
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>
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. |
Prueba bajando el valor de "Wrap at" a 70. |
library/sndhdr.po
Outdated
@@ -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" |
There was a problem hiding this comment.
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".
There was a problem hiding this comment.
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.
Probaré dejando el Wrap como estaba originalmente. |
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. |
There was a problem hiding this 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>
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 ;) |
Muchas gracias @enrique-zarate 🎉 🎉 |
Closes #811