-
Notifications
You must be signed in to change notification settings - Fork 396
Traducido archivo distutils/extending.po #1068
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
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 Sofia, la traducción esta muy bien, te dejo algunas sugerencias a ver que te parecen, una de ellas encaminada a corregir el único error que tiene el build. Todas las sugerencias tienen en cuenta powrap para que si las aceptas no rompan el build y tengas que pasarlo.
@fjsevilla-dev Hola! Te hago una consulta ya que es la primera vez que estoy haciendo un PR (no aceptado por mi) y quiero saber cómo es la mejor manera de resolver sugerencias que mencionaste. ¿Voy modificando el file en mi local con todas las sugerencias, hago push y luego marco las conversaciones como solved ó tengo que hacer click en commit suggestion? |
Hola Sofia, como más cómodo te parezca, no hay ningún problema con eso. Si lo haces directamente a través de GitHub te recomiendo ir a Files changed y usar el botón "Add suggestion to batch" en las sugerencias que aceptes y luego realizar el commit (así las aplicas todas en un solo commit). Si quieres hacerlo desde local no hay problema alguno, solo acuérdate de marcar las sugerencias que compruebes como resueltas aquí como bien comentas. Una ventaja de hacerlo así es que puedes usar precommit, pospell, powrap, etc por si algo se ha escapado antes de aplicar el commit. |
Listo, ya he realizado los cambios en mi local y he hecho push. Marco las conversaciones como resolved. Gracias por la ayuda! |
Actualicé el conflicto del archivo |
@cmaureir Perfecto! Ahora cuando aprueben la review ya estaría merged el PR? |
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.
Listo por mi parte. Felicidades por la traducción @sofiacarballo 🎉🎉🎉
Gracias por la traducción @sofiacarballo 🎉 🎉 |
@cmaureir @fjsevilla-dev muchas gracias!! 😬 voy a probar con otras traducciones más breves que he visto así ya las pueden cerrar y de paso experimento un poquito más con el tema de los issues |
Closes #781