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 library/array#187

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

Merged
cmaureir merged 11 commits intopython:3.8fromalbertcalv:traduccion-library/array
Aug 14, 2020

Conversation

albertcalv
Copy link
Contributor

@albertcalvalbertcalv commentedMay 9, 2020
edited by humitos
Loading

Añadida traducción de library/array.po

Closes#157

@raulcd
Copy link
Collaborator

Hola! Muchas gracias por la PR!!! Hay algunos fallos en Travis debido a algunas palabras que no se encuentran en el diccionario y a algunos links de restructured text mal formados, probablemente falte algún símbolo `
Puedes ver los errores aquí:
https://travis-ci.org/github/PyCampES/python-docs-es/builds/685100496
Muchas gracias

@albertcalv
Copy link
ContributorAuthor

muchas gracias ahora lo reviso :)

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.

Hola@albertcalv , muchas gracias por tu traducción!!! hiciste un montón. Te he hecho algunas sugerencias, algunas son relacionadas con el inglés, por clarificación del contenido, y otras porque hay algunos problemas de formato que pueden saltar más tarde en los builds. También tengo algunas dudas sobre términos pero he puesto menciones a otros para que puedan aportarnos también su punto de vista.

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, ¡AMBOS!

@albertcalv Por favor, cuando puedas, mira las sugerencias de@clacri y luego podemos hacer merge de la PR.

Muchas gracias.

@albertcalv
Copy link
ContributorAuthor

perfecto, muchas gracias por los comentarios, muchas palabras y terminos tenia mis dudas : )

Copy link
ContributorAuthor

@albertcalvalbertcalv left a comment

Choose a reason for hiding this comment

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

Cambios revisados

@clacri
Copy link
Contributor

hola@albertcalv, antes de volver a revisar el contenido, sería genial conseguir que funcionara lo del build.
Parece que el problema principal con las palabras que no pasan el check de pospell y no están en nuestro diccionario de excepciones. Para solucionarlo, deberías revisarlas e incluir las que no sean errores sino excepciones a nuestro fichero dict (lo encuentras en la raíz de tu fork del repo). Las palabras que te fallan (puedes ver esto dando a los Details del sitio donde pone que Travis falló) son:
library/array.po:30:char
library/array.po:30:short
library/array.po:30:double
library/array.po:30:long
library/array.po:30:long
library/array.po:30:long
library/array.po:70:char
library/array.po:90:char
library/array.po:120:short
library/array.po:128:short
library/array.po:152:long
library/array.po:165:long
library/array.po:173:long
library/array.po:173:long
library/array.po:186:long
library/array.po:186:long
library/array.po:243:inicializados
library/array.po:243:term
library/array.po:243:like
library/array.po:253:string
library/array.po:253:inicializador
library/array.po:253:meth
library/array.po:253:inicializador
library/array.po:273:string
library/array.po:277:indexación
library/array.po:277:interificie
library/array.po:277:buffer
library/array.po:298:typecode
library/array.po:310:buffer
library/array.po:310:buffer
library/array.po:310:buffer
library/array.po:310:info
library/array.po:310:itemsize
library/array.po:310:interficies
library/array.po:330:interficies
library/array.po:330:buffer
library/array.po:330:recompatibilidad
library/array.po:330:interficies
library/array.po:330:buffer
library/array.po:344:Byteswap
library/array.po:360:iterabel
library/array.po:360:i
library/array.po:373:string
library/array.po:383:meth
library/array.po:383:frombytes
library/array.po:387:term
library/array.po:412:string
library/array.po:412:frombytes
library/array.po:412:unicodestring
library/array.po:412:encode
library/array.po:412:enc
library/array.po:482:meth
library/array.po:486:string
library/array.po:486:tobytes
library/array.po:486:decode
library/array.po:486:enc
library/array.po:486:string
library/array.po:497:string
library/array.po:497:string
library/array.po:497:string

@clacri
Copy link
Contributor

Igualmente, te recomiendo, y si quieres, te puedo ayudar en caso no te funciona a la primera, que te instales pre-commit porque lo tenemos configurado en el repo para hacer estos mismos checks antes del commit. Esta herramientra no te dejará continuar haciendo commit hasta que se pasen 3 checks:

  1. chequea el wrapping (el tamaño de las lineas basicamente) con powrap
  2. revisa con pospell la ortografía comparando contra diccionarios de español y nuestro propio diccionario
  3. reordena alfabeticamente el fichero dict que tenemos para guardar las excepciones que queremos que pasen el pospell

Para poder emplear pre-commit solo necesitas:

  1. instalar pre-commit usando: pip install pre-commit
  2. activar pre-commit en tu fork del repo de la traducción entrando en su directorio raiz y haciendo: pre-commit install

Un saludo y muchas gracias por tu trabajo! Era una traducción larguísima!!! :100

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.

Hola@albertcalv aún quedan algunas palabras por revisar que te dan fallo en el build, y parece que algunos términos de los que tienen comillas no están cerrados y también producen errores. ¿Puedes revisártelo? Si necesitas ayuda comentanos. Recuerda irte haciendo pull de 3.8 para estar actualizado.

@cmaureir
Copy link
Collaborator

@albertcalv si necesitas ayuda con algo, nos puedes decir :)

@cmaureir
Copy link
Collaborator

@albertcalv resolvi el conflicto de los TRANSLATORS, avisame si necesitas ayuda con algo.

@cmaureir
Copy link
Collaborator

@albertcalv estamos en un sprint este finde, avisanos si podemos darte una mano con este PR!

@cmaureir
Copy link
Collaborator

@albertcalv si no puedes continuar con el trabajo, avísanos así podemos terminar lo que falta.

@albertcalv
Copy link
ContributorAuthor

Hola, si mejor, he estado con otros proyectos estos dias, basicamente esta todo traducido pero faltan corregir errores marcados por el pre-commit

cmaureir reacted with thumbs up emoji

Copy link
Contributor

@narvmtznarvmtz left a comment

Choose a reason for hiding this comment

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

¡Muchas gracias@albertcalv por la traducción y@cmaureir por la revisión y retomar el archivo! 🎉

Deje varias sugerencias de palabras que tenemos estandarizadas y un par de arreglos en otras para que tuviese un poco más sentido en esas entradas. Creo que después de resolver estas sugerencias y que pase los checks estaríamos listos con este archivo :)

cmaureir reacted with thumbs up emoji
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Copy link
Contributor

@narvmtznarvmtz left a comment

Choose a reason for hiding this comment

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

Agregué los espacios que faltan y la palabra adicional al dict y con eso debería pasar pospell, sólo faltaría pasarle powrap y quedaría listo para merge :)

cmaureir reacted with thumbs up emoji
cmaureirand others added2 commitsAugust 14, 2020 19:45
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Copy link
Contributor

@narvmtznarvmtz left a comment

Choose a reason for hiding this comment

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

Se me pasó este cambio.

Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
@cmaureir
Copy link
Collaborator

@narvmtz espero tu approve

@narvmtz
Copy link
Contributor

¡Muchas gracias@cmaureir! gran trabajo con este archivo, ya queda solo hacer merge 🎉

@cmaureircmaureir merged commit71cbaac intopython:3.8Aug 14, 2020
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@humitoshumitoshumitos left review comments

@clacriclacriclacri requested changes

@cmaureircmaureircmaureir approved these changes

@narvmtznarvmtznarvmtz approved these changes

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

Successfully merging this pull request may close these issues.

Translatelibrary/array.po
6 participants
@albertcalv@raulcd@clacri@cmaureir@narvmtz@humitos

[8]ページ先頭

©2009-2025 Movatter.jp