8000 Traducido archivo library/array by albertcalv · Pull Request #187 · python/python-docs-es · GitHub
[go: up one dir, main page]
Skip to content

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 11 commits into from
Aug 14, 2020
Merged

Traducido archivo library/array #187

merged 11 commits into from
Aug 14, 2020

Conversation

albertcalv
Copy link
Contributor
@albertcalv albertcalv commented May 9, 2020 •
edited by humitos
Loading

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

Closes #157

@raulcd
Copy link
8000
Collaborator
raulcd commented May 9, 2020

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
Contributor Author

muchas gracias ahora lo reviso :)

clacri
clacri suggested changes May 11, 2020
View reviewed changes
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.

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.

8000
humitos
humitos reviewed May 12, 2020
View reviewed changes
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, ¡AMBOS!

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

Muchas gracias.

@albertcalv
Copy link
Contributor Author

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

@albertcalv albertcalv closed this May 13, 2020
@albertcalv albertcalv reopened this May 13, 2020
@albertcalv albertcalv closed this May 13, 2020
@albertcalv albertcalv reopened this May 13, 2020
albertcalv
albertcalv commented May 14, 2020
View reviewed changes
Copy link
Contributor Author
@albertcalv albertcalv 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
clacri commented May 17, 2020

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
clacri commented May 17, 2020

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

clacri
clacri reviewed May 24, 2020
View reviewed changes
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.

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
cmaureir commented Jul 5, 2020

@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
Contributor Author

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

narvmtz
narvmtz suggested changes Aug 14, 2020
View reviewed changes
Copy link
Contributor
@narvmtz narvmtz 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 @narvmtz
Apply suggestions from code review
bd92d1f 
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
narvmtz
narvmtz suggested changes Aug 14, 2020
View reviewed changes
Copy link
Contributor
@narvmtz narvmtz 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 and others added 2 commits August 14, 2020 19:45
@cmaureir @narvmtz
Apply suggestions from code review
a70f796 
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
@cmaureir
powrap library/array
71490db
narvmtz
narvmtz suggested changes Aug 14, 2020
View reviewed changes
Copy link
Contributor
@narvmtz narvmtz 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.

@cmaureir @narvmtz
Apply suggestions from code review
d843cc4 
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
cmaureir
cmaureir reviewed Aug 14, 2020
View reviewed changes
@cmaureir
Copy link
Collaborator

@narvmtz espero tu approve

@narvmtz
Copy link
Contributor
narvmtz commented Aug 14, 2020

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

@cmaureir cmaureir merged commit 71cbaac into python:3.8 Aug 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@humitos humitos humitos left review comments

@clacri clacri clacri requested changes

@cmaureir cmaureir cmaureir approved these changes

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

Translate library/array.po
6 participants
@albertcalv @raulcd @clacri @cmaureir @narvmtz @humitos
0