-
Notifications
You must be signed in to change notification settings - Fork 396
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
Traducción archivo 'library/email.contentmanager.po' #1167
Conversation
There was a problem hiding this 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.
Listo, @alcides29 ya lo corregí 😄 |
There was a problem hiding this 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.
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 :) |
No estoy muy seguro, vamos a consultar si otra persona nos puede dar una mano con esto. |
Creo que ya lo solucioné |
Ok, hace un commit nuevamente cuando puedas para que procese el Travis |
adc55e6
to
c52bb04
Compare
There was a problem hiding this 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 😄
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 |
Muchas gracias @Yenniferh por la traducción! 🎉 |
Closes #816