Skip to content

Traducido archivo {zipimport} #125

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

Closed
wants to merge 18 commits into from
Closed

Traducido archivo {zipimport} #125

wants to merge 18 commits into from

Conversation

qagustina
Copy link
Contributor

aclaraciones:
"hooks" agrege como "gancho"
"subpath" como "subruta o sub-ruta", no estaba segura
"adds" = "añade" me parecio más formal
"zipimporter Objets" = "objetos zipimporter", al poner en mayúscula la primer letra me daba error
"raised" agrege como "generado"

Copy link
Collaborator

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

Hola! Está perfecto, el build falla por un par de cosas que no encuentra en el diccionario. Creo que la mayoría se pueden añadir al archivo dict que contiene las excepciones:

Would rewrap:
- library/zipimport.po
The command "powrap --check --quiet **/*.po" exited with 1.
84.10s$ pospell -p dict -l es_AR -l es_ES **/*.po
library/zipimport.po:27:Zip
library/zipimport.po:83:Phil
library/zipimport.po:83:Katz
library/zipimport.po:91:Zip
library/zipimport.po:95:Ahlstrom
library/zipimport.po:95:Just
library/zipimport.po:95:Rossum
library/zipimport.po:121:zipimporter
library/zipimport.po:121:tambíen
library/zipimport.po:130:zipimporter
library/zipimport.po:138:zipimporter
library/zipimport.po:160:zipimporter
library/zipimport.po:160:ignoradoestá
library/zipimport.po:236:zipimporter
The command "pospell -p dict -l es_AR -l es_ES **/*.po" exited with 255.

Las que creo que se pueden añadir son:
Zip, Phil, Katz, Ahlstrom, Just, Rossum
para zipimporter quizás podemos cambiarlo por la clase para que no se queje:

:class:`zipimporter`

hay que cambiar tambíen por también
y el caso de ignoradoestá es el que aparece en el texto como ignorado---está podemos separarlo con punto y coma para que no se queje: ignorado; está

@humitos
Copy link
Collaborator

humitos commented May 7, 2020

No he visto el original (estoy al teléfono) , pero si "zipimporter" no está con :class: no deberíamos ponerlo con class en la traducción. Eso va a cambiar el render y generar links donde no van.

@qagustina
Copy link
Contributor Author

hice otro commit modificando:
tambíen por también
ignorado---está por ignorado; está
para zipimporter y los nombres como Katz, Phil, Ahlstrom, Just, Rossum deje tal cual estaba

Copy link
Collaborator

@humitos humitos left a comment

Choose a reason for hiding this comment

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

Te había quedado un problema con el merge. Hice una sugerencia para solucionarlo. La voy a aceptar y eso va a volver a correr las pruebas, ya que están fallando actualmente por esto.

@clacri
Copy link
Contributor

clacri commented May 12, 2020

Hola @qagustina !!!! 🥳
parece que el build sigue fallando en los dos chequeos, tanto en el del tamaño de las líneas como en el del diccionario. Para solucionar el primero, el de powrap, lo que necesitas es (si lo tienes instalado, sino, se puede instalar con pip), ejecutar
powrap path_a_tu_po_file.po
esto te cambiará las líneas automáticamente
Para el segundo check, necesitas incluir en nuestro fichero dict (lo encuentras en el root directory del repo) las palabras que van a ser excepciones, que en tu caso son:
Zip
Phil
Katz
Ahlstrom
Just
Rossum
y bueno zipimporter como te dijo @humitos o lo puedes incluir o bien le pones el tag de class.

Bueno, si necesitas cualquier cosa para ayudarte a finalizar el PR, dime. Un saludo :)

@qagustina
Copy link
Contributor Author

Hola @clacri muchas gracias por la explicación! estoy siguiendo los pasos.
Te escribo a telegram :)

Copy link
Contributor

@clacri clacri left a comment

Choose a reason for hiding this comment

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

he resuelto el conflicto, a ver si ahora funciona

Co-authored-by: Claudia Millán <clmilneb@gmail.com>
@clacri
Copy link
Contributor

clacri commented May 13, 2020

vuelve a dar los errores del comienzo, sobre las palabras que no hay en dict y sobre el powrap. Lo del powrap si estás trabajando en windows podria ser por los finales de linea (ya nos ha pasado). No entiendo porque se te sobreescribió lo del diccionario, pero me parece que el problema es similar a lo que os pasó hace unos días. Quizás @humitos pueda ayudarte mejor que yo

Copy link
Collaborator

@humitos humitos 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! Hice unas pequeñas correcciones 😄

qagustina and others added 7 commits May 13, 2020 13:36
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
la primer letra había dejado en minúscula porque el poedit me lo marcaba como error, solo eso.

Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
@humitos
Copy link
Collaborator

humitos commented May 14, 2020

Cierro esta ya que se hizo merge de #266! Gracias gente!

@humitos humitos closed this May 14, 2020
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.

4 participants