Skip to content

Traducción csv.po (100%)#377

Merged
cmaureir merged 14 commits into
python:3.8from
marcorichetta:traduccion-csv
Jul 28, 2020
Merged

Traducción csv.po (100%)#377
cmaureir merged 14 commits into
python:3.8from
marcorichetta:traduccion-csv

Conversation

@marcorichetta
Copy link
Copy Markdown
Contributor

@marcorichetta marcorichetta commented May 27, 2020

Todavía tengo la versión desactualizada del pre-commit y veo que me "ordenó" csv como quiso.
Ahora entiendo porque lo sacaron 😆

Closes #191

Copy link
Copy Markdown
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.

Muchas gracias por tu trabajo! Te hice una pequeña review de lo que has hecho hasta el momento. Viene muy bien!

Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
@marcorichetta
Copy link
Copy Markdown
Contributor Author

Muchas gracias por tu trabajo! Te hice una pequeña review de lo que has hecho hasta el momento. Viene muy bien!

Gracias a vos por la review y por el apoyo a pesar del delay 😆

Copy link
Copy Markdown
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

un detalle pequeño que encontre :)

Comment thread library/csv.po Outdated
@cmaureir
Copy link
Copy Markdown
Collaborator

te resolví el merge del dict @marcorichetta a ver que va diciendo travis :)

Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@marcorichetta marcorichetta changed the title Traducción inicial csv.po (13%) Traducción inicial csv.po (50%) Jul 22, 2020
@marcorichetta
Copy link
Copy Markdown
Contributor Author

Muchas gracias @cmaureir!

Parece que necesito agregar estas palabras al dict.

$ pospell -p dict.txt -l es_AR -l es_ES **/*.po
library/csv.po:287:com
library/csv.po:344:tabulaciones
library/csv.po:426:caracter
The command "pospell -p dict.txt -l es_AR -l es_ES **/*.po" exited with 255.

Copy link
Copy Markdown
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

creo que solo tabulaciones deberia ser agregada

Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@cmaureir
Copy link
Copy Markdown
Collaborator

Aparte de agregar tabulaciones hay que pasarle powrap al archivo y Travis no se va a quejar :)

@marcorichetta
Copy link
Copy Markdown
Contributor Author

Aparte de agregar tabulaciones hay que pasarle powrap al archivo y Travis no se va a quejar :)

Lo agregaré en mi dict local. Respecto al powrap no recuerdo porqué tenía desinstalado el pre-commit pero ya lo activé asi que con suerte atrapo los errores antes jaja

Gracias de nuevo @cmaureir!

@marcorichetta marcorichetta changed the title Traducción inicial csv.po (50%) Traducción inicial csv.po (59%) Jul 25, 2020
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
@cmaureir
Copy link
Copy Markdown
Collaborator

@marcorichetta que te parecen los comentarios que dejé? asi nos acercamos más al merge 🗡️

Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@marcorichetta
Copy link
Copy Markdown
Contributor Author

@cmaureir Muchas gracias por las correcciones. Voy a tener que empezar a buscar mejor en la memoria de traducción así no se me pasan esas palabras 😆

@cmaureir
Copy link
Copy Markdown
Collaborator

A mi también se me olvidan! jaja @marcorichetta
Falta solo pasarle 'powrap' al archivo localmente y el build va a pasar.
Recuerda en cambiarlo de draft cuando termines el archivo completo :)

@marcorichetta marcorichetta changed the title Traducción inicial csv.po (59%) Traducción csv.po (100%) Jul 28, 2020
@marcorichetta
Copy link
Copy Markdown
Contributor Author

@cmaureir Casi hago un desastre con git commit-fetch-pull-etc 🤦 pero creo que al final los cambios se subieron bien.

@marcorichetta marcorichetta marked this pull request as ready for review July 28, 2020 04:54
Copy link
Copy Markdown
Collaborator

@cmaureir cmaureir left a comment

Choose a reason for hiding this comment

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

Un par de detallitos @marcorichetta :)

Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
Comment thread library/csv.po Outdated
marcorichetta and others added 2 commits July 28, 2020 10:42
Devolver => Retornar

Co-authored-by: Cristián Maureira-Fredes <[email protected]>
@cmaureir
Copy link
Copy Markdown
Collaborator

cmaureir commented Jul 28, 2020

Gracias por todo tu trabajo @marcorichetta 🎉 🎉
Si en el futuro encuentras algún problemilla que se nos haya pasado, siéntete libre en enviar algún PR que lo arregle ;)

@cmaureir cmaureir merged commit 83254cf into python:3.8 Jul 28, 2020
@marcorichetta marcorichetta deleted the traduccion-csv branch July 28, 2020 20:39
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/csv.po

3 participants