Skip to content

Traduccion library distutils #1558

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

Closed
wants to merge 2 commits into from

Conversation

lis17
Copy link
Contributor

@lis17 lis17 commented Oct 20, 2021

Primera corrección #1433

@cmaureir
Copy link
Collaborator

Hola @lis17 lamentablemente te trajiste contigo los cambios de tu otro Pull Request.
¿Qué paso?

En tu fork, hiciste tu primera rama relacionada a graphlib.
Como el PR fue invalido, te buscaste otra issue y comenzaste a trabajar en ella,
el problema es que creaste una rama con otro nombre sin volver a la rama principal 3.10
Entonces en vez de tener

Rama 3.10 -> rama graphlib
Rama 3.10 -> rama distutils

Ahora tienes

Rama 3.10 -> rama graphlib -> rama distutils

Con lo que los cambios incorrectos de graphlib están acá.

Cuando trabajes haciendo PRs a otros proyectos, siempre vuelve a la rama principal, actualiza, y comienza una nueva rama.

¿Como solucionamos esto?

Creo que para hacerlo más facil y no complicarte con cosas de git,
te recomiendo que guardes el párrafo que tradujiste en otro lugar,
y localmente en tu repositorio hagas lo siguiente

git checkout 3.10   # volver a la rama principal
git checkout -b traduccion-distutils-2  # creas otra rama con un nombre similar
<vuelves a traducir lo que hiciste acá en el archivo library/distutils.po>
git add library/distutils.po
git commit -m "Traducción library/distutils"
git push origin traduccion-disutils-2

Con eso abriras otro Pull Request y así cerramos este.

@cmaureir cmaureir closed this Oct 21, 2021
@cmaureir
Copy link
Collaborator

reemplazado por #1562

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.

2 participants