Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
qagustina wants to merge18 commits intopython:3.8fromqagustina:traduccion_zipimport
Closed

Traducido archivo {zipimport}#125

qagustina wants to merge18 commits intopython:3.8fromqagustina:traduccion_zipimport

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

@raulcdraulcd 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 archivodict que contiene las excepciones:

Would rewrap:- library/zipimport.poThe command "powrap --check --quiet **/*.po" exited with 1.84.10s$ pospell -p dict -l es_AR -l es_ES **/*.polibrary/zipimport.po:27:Ziplibrary/zipimport.po:83:Phillibrary/zipimport.po:83:Katzlibrary/zipimport.po:91:Ziplibrary/zipimport.po:95:Ahlstromlibrary/zipimport.po:95:Justlibrary/zipimport.po:95:Rossumlibrary/zipimport.po:121:zipimporterlibrary/zipimport.po:121:tambíenlibrary/zipimport.po:130:zipimporterlibrary/zipimport.po:138:zipimporterlibrary/zipimport.po:160:zipimporterlibrary/zipimport.po:160:ignoradoestálibrary/zipimport.po:236:zipimporterThe 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
parazipimporter quizás podemos cambiarlo por la clase para que no se queje:

:class:`zipimporter`

hay que cambiartambíen portambién
y el caso deignoradoestá es el que aparece en el texto comoignorado---está podemos separarlo con punto y coma para que no se queje:ignorado; está

@humitos
Copy link
Collaborator

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
ContributorAuthor

hice otro commit modificando:
tambíen portambién
ignorado---está porignorado; está
parazipimporter y los nombres comoKatz, Phil, Ahlstrom, Just, Rossum deje tal cual estaba

Copy link
Collaborator

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

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 reacted with thumbs up emoji

@qagustina
Copy link
ContributorAuthor

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

Copy link
Contributor

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

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

@humitoshumitos 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 😄

qagustinaand others added7 commitsMay 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

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

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@humitoshumitoshumitos left review comments

@raulcdraulcdraulcd left review comments

@clacriclacriclacri requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@qagustina@humitos@clacri@raulcd

[8]ページ先頭

©2009-2025 Movatter.jp