Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
cmaureir merged 4 commits intopython:3.8fromsofiacarballo:traduccion-extending.co
Oct 14, 2020
Merged

Traducido archivo distutils/extending.po#1068

cmaureir merged 4 commits intopython:3.8fromsofiacarballo:traduccion-extending.co
Oct 14, 2020

Conversation

sofiacarballo
Copy link
Contributor

Closes#781

Copy link
Contributor

@fjsevilla-devfjsevilla-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
ContributorAuthor

sofiacarballo commentedOct 13, 2020
edited
Loading

@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 aFiles 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
ContributorAuthor

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

fjsevilla-dev reacted with thumbs up emoji

@cmaureir
Copy link
Collaborator

Actualicé el conflicto del archivoTRANSLATORS

sofiacarballo and fjsevilla-dev reacted with thumbs up emoji

@sofiacarballo
Copy link
ContributorAuthor

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

Copy link
Contributor

@fjsevilla-devfjsevilla-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 🎉🎉🎉

@cmaureircmaureir merged commit90aeadb intopython:3.8Oct 14, 2020
@cmaureir
Copy link
Collaborator

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

@sofiacarballo
Copy link
ContributorAuthor

@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 freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@cmaureircmaureircmaureir approved these changes

@fjsevilla-devfjsevilla-devfjsevilla-dev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Translate 'distutils/extending.po'
3 participants
@sofiacarballo@fjsevilla-dev@cmaureir

[8]ページ先頭

©2009-2025 Movatter.jp