Skip to content

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

Merged
merged 4 commits into from
Oct 14, 2020
Merged

Traducido archivo distutils/extending.po #1068

merged 4 commits into from
Oct 14, 2020

Conversation

sofiacarballo
Copy link
Contributor

Closes #781

Copy link
Contributor

@fjsevilla-dev fjsevilla-dev left a 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.

@sofiacarballo
Copy link
Contributor Author

sofiacarballo commented Oct 13, 2020

@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?
Gracias!

@fjsevilla-dev
Copy link
Contributor

@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 los conflictos 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?
Gracias!

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.

@sofiacarballo
Copy link
Contributor Author

Listo, ya he realizado los cambios en mi local y he hecho push. Marco las conversaciones como resolved. Gracias por la ayuda!

@cmaureir
Copy link
Collaborator

Actualicé el conflicto del archivo TRANSLATORS

@sofiacarballo
Copy link
Contributor Author

@cmaureir Perfecto! Ahora cuando aprueben la review ya estaría merged el PR?

Copy link
Contributor

@fjsevilla-dev fjsevilla-dev left a 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 🎉🎉🎉

@cmaureir cmaureir merged commit 90aeadb into python:3.8 Oct 14, 2020
@cmaureir
Copy link
Collaborator

Gracias por la traducción @sofiacarballo 🎉 🎉
y gracias por la revisión @fjsevilla-dev 👯

@sofiacarballo
Copy link
Contributor Author

@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

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 'distutils/extending.po'
3 participants