8000 Traduccion library/email by wilo-dev · Pull Request #1081 · python/python-docs-es · GitHub
[go: up one dir, main page]
Skip to content

Traduccion library/email #1081

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
Closed
wants to merge 20 commits into from
Closed

Traduccion library/email #1081

wants to merge 20 commits into from

Conversation

wilo-dev
Copy link
Contributor

close #847

cmaureir
cmaureir requested changes Oct 15, 2020
View reviewed changes
8000 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.

Gracias por tu traducción, pequeño detalle

@cmaureir
Copy link
Collaborator

Si miras el CI, hay un par de cosas que resolver, particularmente:

Would rewrap:
- library/email.po
The command "powrap --check --quiet **/*.po" exited with 1.
0.01s$ awk 1 dict dictionaries/*.txt > dict.txt
The command "awk 1 dict dictionaries/*.txt > dict.txt" exited with 0.

98.37s$ pospell -p dict.txt -l es_AR -l es_ES **/*.po
library/email.po:59:subcomponentes
library/email.po:83:aoquete
library/email.po:103:instanciando
library/email.po:103:serializarlo
library/email.po:178:subcomponentes
The command "pospell -p dict.txt -l es_AR -l es_ES **/*.po" exited with 255.

las palabras que estén ortográficamente correctas debes agregarlas a un diccionario nuevo en dictionaries/library_email.txt agregando una palabra por línea.

Luego del proceso, tienes que aplicar powrap al archivo library/email.po.

Más información en https://python-docs-es.readthedocs.io/es/3.8/CONTRIBUTING.html

@cmaureir cmaureir mentioned this pull request Oct 15, 2020
Closed
wilo-dev and others added 3 commits October 15, 2020 19:20
@wilo-dev @cmaureir
Apply suggestions from code review
2496b2f 
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
@wilo-dev
Correcion y powrap library/email
3b70d60
@wilo-dev
was added library_email.txt
86b8092
@cmaureir
Copy link
Collaborator

Sigue faltando powrap y hay unos warnings:

cpython/Doc/library/email.rst:37: WARNING: inconsistent term references in translated message. original: [':mod:`~email.parser`', ':mod:`~email.generator`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`'], translated: [':mod:`~email.parser`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`']

cpython/Doc/library/email.rst:46: WARNING: inconsistent term references in translated message. original: [':mod:`~email.policy`', ':class:`~email.message.EmailMessage`', ':mod:`~email.generator`', ':mod:`~email.parser`', ':mod:`~email.policy`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`', ':mod:`~email.parser`', ':mod:`~email.generator`'], translated: [':mod:`~email.generator`', ':mod:`~email.parser`', ':mod:`~email.policy`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`', ':mod:`~email.parser`', ':mod:`~email.generator`']

Fíjate en la entrada donde pone email.rst:37 que no es la línea 37 del archivo .po

@cmaureir
Copy link
Collaborator

Extraño, siguen apareciendo un par de errores y powrap:

cpython/Doc/library/email.rst:37: WARNING: inconsistent term references in translated message. original: [':mod:`~email.parser`', ':mod:`~email.generator`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`'], translated: [':mod:`~email.parser`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`']

cpython/Doc/library/email.rst:46: WARNING: inconsistent term references in translated message. original: [':mod:`~email.policy`', ':class:`~email.message.EmailMessage`', ':mod:`~email.generator`', ':mod:`~email.parser`', ':mod:`~email.policy`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`', ':mod:`~email.parser`', ':mod:`~email.generator`'], translated: [':mod:`~email.policy`', ':class:`~email.message.EmailMessage`', ':mod:`~email.generator`', ':mod:`~email.parser`', ':mo
8000
d:`~email.policy`', ':class:`~email.message.EmailMessage`', ':class:`~email.message.EmailMessage`', ':mod:`~email.generator`']

cmaureir
cmaureir requested changes Oct 17, 2020
View reviewed changes
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.

eso deberia solucionar los warnings

cmaureir
cmaureir requested changes Oct 17, 2020
View reviewed changes
reference/toplevel_components.po
"Si bien una especificación de lenguaje no necesita prescribir cómo se invoca "
"al intérprete de lenguaje, es útil tener una noción de un programa completo "
"de Python. Un programa completo de Python se ejecuta en un entorno "
"mínimamente inicializado: todos los módulos estándar e integrados están "
Copy link
Collaborator

Choose a reason for hiding this comment

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

creo que hay que agregar "mínimamente" a un nuevo archivo dictionaries/reference_toplevel_components.txt

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Saludos m podría ayudar mencionando como puedo generar los warnings, estoy usando visual studio code

@cmaureir
Copy link
Collaborator

@willian593 tienes los warning acá mismo en Travis:
travis

@cmaureir
Copy link
Collaborator

@willian593 creo que estás teniendo muchos problemas con haber comenzado dos PR al mismo tiempo.
Ahora tienes todo el contenido de tu otro PR acá también.

Screenshot_2020-10-19 Traduccion library email by willian593 · Pull Request #1081 · python python-docs-es

Por eso insistí en que trabajaras en uno solo.

El flujo de trabajo hubiera sido:

git checkout 3.8  # rama principal
git checkout -b rama_local_1  # Rama para un PR
# Una vez realizado los cambios y enviado el PR a github
git checkout 3.8  # vuelves a la rama principal
git checkout -b rama_local_2  # Creas otra rama independiente

El problema que tienes es que has hecho la rama "2" dentro de la rama "1".

Te recomiendo que puedas copiar ambas traducciones a un lugar seguro, cierres ambos PRs y los envies de a uno.

@wilo-dev wilo-dev closed this Oct 19, 2020
@wilo-dev
Copy link
Contributor Author

Saludos.
Tengo una pregunta, también tengo que eliminar la rama del repositorio remoto y después de haber cerrado el PR tengo que hacerme un nuevo git clone o puedo seguir trabajado con el que ya tengo

@cmaureir
Copy link
Collaborator

solo crear una nueva rama, no es necesario hacer otro fork ni nada:

# Respalda tu archivo traducido a un lugar seguro
git checkout 3.8  # vuelves a la rama principal
git checkout -b traduccion-library-email2  # u otro nombre que quieras
# copias tu archivo de vuelta dentro del directorio library
git add library/email.po
git commit -m "Traducido library/email"
git push origin traduccion-library-email2 # o el nombre que le hayas dado.

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

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

Translate 'library/email.po'
2 participants
@wilo-dev @cmaureir
0