Skip to content

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

Merged
merged 6 commits into from
Oct 5, 2020
Merged

Traducción library/colorsys #948

merged 6 commits into from
Oct 5, 2020

Conversation

iracaballero
Copy link
Contributor

Closes #813

Copy link

@Bgeninatti Bgeninatti left a 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 😄

iracaballero and others added 4 commits October 5, 2020 16:44
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Co-authored-by: Bruno <brunogeninatti@gmail.com>
Copy link

@Bgeninatti Bgeninatti left a 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!

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

Choose a reason for hiding this comment

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

Suggested change
"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

Copy link
Contributor Author

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!

@iracaballero
Copy link
Contributor Author

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

@Bgeninatti
Copy link

👏 🎉 Muy bien @iracaballero !

Muchas gracias por tu aporte y felicitaciones.

Ya podemos hacer merge de este PR @cmaureir

@cmaureir cmaureir merged commit 2449e18 into python:3.8 Oct 5, 2020
@cmaureir
Copy link
Collaborator

cmaureir commented Oct 5, 2020

🎉 🎉

@iracaballero iracaballero deleted the traduccion-colorsys branch October 5, 2020 17:06
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/colorsys.po'
3 participants