Skip to content

Translate library/poplib.po #764

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 9 commits into from
Sep 26, 2020
Merged

Translate library/poplib.po #764

merged 9 commits into from
Sep 26, 2020

Conversation

fenriquegimenez-zz
Copy link
Contributor

@fenriquegimenez-zz fenriquegimenez-zz commented Sep 19, 2020

Espero que sea de utilidad, aguardo feedback!
Closes #690

@fenriquegimenez-zz fenriquegimenez-zz changed the title Translate library/poplib.po #690 Translate library/poplib.po Sep 19, 2020
@cmaureir
Copy link
Collaborator

Hola @fenriquegimenez, muchas gracias por la traducción!.
Si miras el el estado de Travis CI, algo ha fallado. Si clickas en 'Details', y luego 'The Build' verás los problemas,
los que vi son:

  • Tienes que aplicar powrap a tu archivo, es decir, powrap library/poplib.po y luego agregarlo y hacer otro push en este mismo PR (no hay que crear otro PR!)
  • Hay un par de warnings cuando uno no pone un espacio en blanco después de un , en ese caso es: "``STLS``introducida" y ":mod:socket`no "
  • hay unas palabras que no reconoce pospell, con lo que tienes que ver si están mal escritas o si vale la pena agregarlas a un nuevo archivo que se llame dictionaries/library_poplib.txt, puedes mirar los otros que hay dentro de ese directorio para ver como son.

Me cuentas como va todo!
Son detalles pequeños, así que de seguro podremos hacer el merge pronto 👍

@fenriquegimenez-zz
Copy link
Contributor Author

fenriquegimenez-zz commented Sep 25, 2020

Hola @cmaureir, desde ya disculpas por la cantidad de commits que hay en este pull request, tuve el inconveniente de no poder pasar el check de Travis CI, y luego de varios intentos pude dar finalmente con la solución (pasar el archivo traducido al script powrap (en linux, lo estaba haciendo en windows y no me funcionó, ni idea del porqué).
Espero que esté todo ok, y aguardo comentarios.
Saludos!

Copy link
Contributor

@eamanu eamanu left a comment

Choose a reason for hiding this comment

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

Lo veo bien

@cmaureir cmaureir merged commit 5633927 into python:3.8 Sep 26, 2020
@cmaureir
Copy link
Collaborator

Muchas gracias @fenriquegimenez por la traducción! y gracias @eamanu por la revisión 🎉

@@ -1454,3 +1454,20 @@ reintrodujo
radix
léxicamente
guión
Eudora
Copy link
Collaborator

Choose a reason for hiding this comment

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

no me di cuenta que se pasaron estas palabras, ahora las borro.

@fenriquegimenez-zz fenriquegimenez-zz deleted the traduccion-poplib branch September 27, 2020 03:43
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/poplib.po
4 participants