Skip to content

Traducción archivo 'library/email.contentmanager.po'#1167

Merged
cmaureir merged 4 commits into
python:3.8from
Yenniferh:traduccion-email.contentmanager
Dec 14, 2020
Merged

Traducción archivo 'library/email.contentmanager.po'#1167
cmaureir merged 4 commits into
python:3.8from
Yenniferh:traduccion-email.contentmanager

Conversation

@Yenniferh

Copy link
Copy Markdown
Contributor

Closes #816

@Yenniferh Yenniferh changed the title Traducción archivo email.contentmanager.po Traducción archivo 'library/email.contentmanager.po' Dec 7, 2020
@Yenniferh Yenniferh marked this pull request as ready for review December 8, 2020 02:30

@alcides29 alcides29 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hola @Yenniferh Gracias por tu aporte en la traducción, me parece un muy buen trabajo. solo hay una pequeña modificación que hacerle y que pase nuevamente el validador.

Comment thread library/email.contentmanager.po Outdated
@Yenniferh

Copy link
Copy Markdown
Contributor Author

Listo, @alcides29 ya lo corregí 😄

@alcides29 alcides29 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hola @Yenniferh encontré otra modificación que habría que hacerle. Ahí deje una sugerencia. Por otra parte el travis también encontró un error
The command "powrap --check --quiet **/*.po" exited with 1.

Esto significa que hay que ejecutar el powrap de nuevo, cada vez que se realiza una modificación al archivo para verificar que las lineas no sean tan largas.
git pull origin [nombre de la rama] actualiza tu rama local
powrap library/email.contenmanager.po

Luego hay que hacer un commit de vuelta

git add library/email.contentmanager.po
git commit -m "powrap library/email.contentmanager.po"
git push origin [nombre de la rama]

Ahora se ejecuta el build de nuevo y si no encuentra ningún error ya estará listo para el merge.

Comment thread library/email.contentmanager.po Outdated
@alcides29

Copy link
Copy Markdown
Contributor

También, si querés, podes fijarte siempre si pasa los controles y si hay un error, en el Travis por ejemplo, a lado, en la parte derecha hay un enlace "detalles", al darle click allí y luego abajo mismo "ver más detalles en Travis CI" te mostrará el log de ejecución y si hubo algún error, cuál fue el error para poder corregir :)

Comment thread library/email.contentmanager.po Outdated
@alcides29

Copy link
Copy Markdown
Contributor

Al parecer sigue habiendo un error con el powrap, pero cuando ejecuto powrap library/email.contenmanager.po y luego hago un git status, no hay cambios para commitear. ¿Qué podría estar sucediendo? @alcides29

No estoy muy seguro, vamos a consultar si otra persona nos puede dar una mano con esto.

@Yenniferh

Copy link
Copy Markdown
Contributor Author

Creo que ya lo solucioné

@alcides29

alcides29 commented Dec 12, 2020

Copy link
Copy Markdown
Contributor

Creo que ya lo solucioné

Ok, hace un commit nuevamente cuando puedas para que procese el Travis

@Yenniferh Yenniferh force-pushed the traduccion-email.contentmanager branch from adc55e6 to c52bb04 Compare December 12, 2020 17:30

@alcides29 alcides29 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Genial @Yenniferh Ahora solo hay que esperar a que uno de los administradores haga el merge 😄

@Yenniferh

Copy link
Copy Markdown
Contributor Author

Listo, era que en Windows estaba ejecutando el powrap desde la Git Bash. Probé ejecutandolo desde el cmd y listo, cuando hice git status aparecieron los cambios. Gracias por la revisión @alcides29 y Saludos

@cmaureir cmaureir merged commit b555cee into python:3.8 Dec 14, 2020
@cmaureir

Copy link
Copy Markdown
Collaborator

Muchas gracias @Yenniferh por la traducción! 🎉
y gracias @alcides29 por la revisión :)

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/email.contentmanager.po'

3 participants