Skip to content

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

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 4 commits into from
Dec 14, 2020

Conversation

Yenniferh
Copy link
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
Copy link
Contributor

@alcides29 alcides29 left a comment

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.

@Yenniferh
Copy link
Contributor Author

Listo, @alcides29 ya lo corregí 😄

Copy link
Contributor

@alcides29 alcides29 left a comment

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.

@alcides29
Copy link
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 :)

@alcides29
Copy link
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
Contributor Author

Creo que ya lo solucioné

@alcides29
Copy link
Contributor

alcides29 commented Dec 12, 2020

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
Copy link
Contributor

@alcides29 alcides29 left a comment

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