Traducción archivo 'library/email.contentmanager.po'#1167
Conversation
alcides29
left a comment
There was a problem hiding this comment.
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.
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.
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