-
Notifications
You must be signed in to change notification settings - Fork 396
Traducción library/colorsys #948
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.
Excelente trabajo @iracaballero ! 👏
Te hice algunas correcciones mínimas.
Si te fijas en el build de Travis, está fallando por unas palabras en inglés que quedaron en la traducción en español.
Cuando pasa esto las palabras que quedan en inglés deben ir en cursiva (rodeadas por asteríscos).
Felicitaciones y espero que podamos hacer merge pronto 😄
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
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.
Ya casi lo tenés!
library/colorsys.po
Outdated
@@ -35,42 +37,55 @@ msgid "" | |||
"coordinate is between 0 and 1, but the I and Q coordinates can be positive " | |||
"or negative. In all other spaces, the coordinates are all between 0 and 1." | |||
msgstr "" | |||
"El módulo :mod: `colorsys` define conversiones bidireccionales de valores de " |
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.
"El módulo :mod: `colorsys` define conversiones bidireccionales de valores de " | |
"El módulo :mod:`colorsys` define conversiones bidireccionales de valores de " |
Te quedó un espacio después de :mod:
por eso no pasa el build
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.
Ai, ojalá haberlo leido antes... pero bueno al final me he dado cuenta. Gracias igualmente!
@Bgeninatti he añadido los cambios que habías propuesto, y he puesto tres palabras en inglés entre comillas que faltaban. Me seguía dando un fallo en el test de Travis por un espacio en blanco pero ya lo he resuelto. Espero que esté todo correcto. |
👏 🎉 Muy bien @iracaballero ! Muchas gracias por tu aporte y felicitaciones. Ya podemos hacer merge de este PR @cmaureir |
🎉 🎉 |
Closes #813