Skip to content

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

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

Merged
merged 14 commits into from
Jul 28, 2020
Merged

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

merged 14 commits into from
Jul 28, 2020

Conversation

marcorichetta
Copy link
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
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!

Co-authored-by: Manuel Kaufmann <humitos@gmail.com>
@marcorichetta
Copy link
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
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 :)

@cmaureir
Copy link
Collaborator

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

Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@marcorichetta marcorichetta changed the title Traducción inicial csv.po (13%) Traducción inicial csv.po (50%) Jul 22, 2020
@marcorichetta
Copy link
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
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

Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir
Copy link
Collaborator

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

@marcorichetta
Copy link
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 and others added 2 commits July 25, 2020 05:03
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@marcorichetta marcorichetta changed the title Traducción inicial csv.po (50%) Traducción inicial csv.po (59%) Jul 25, 2020
@cmaureir
Copy link
Collaborator

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

Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@marcorichetta
Copy link
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
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 and others added 2 commits July 28, 2020 01:46
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@marcorichetta marcorichetta changed the title Traducción inicial csv.po (59%) Traducción csv.po (100%) Jul 28, 2020
@marcorichetta
Copy link
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
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 :)

marcorichetta and others added 2 commits July 28, 2020 10:42
Devolver => Retornar

Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@cmaureir
Copy link
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