Skip to content

Traducción reference/datamodel.po #1168

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 1 commit into from
Closed

Traducción reference/datamodel.po #1168

wants to merge 1 commit into from

Conversation

migrmrz
Copy link
Contributor

@migrmrz migrmrz commented Dec 9, 2020

Avance a 35%
Closes #304

@@ -972,7 +973,7 @@ msgid ""
msgstr ""
"Los objetos de función también admiten obtener y establecer atributos "
"arbitrarios, que pueden usarse, por ejemplo, para adjuntar metadatos a "
"funciones. La notación regular de atributo por punto se usa para obtener y "
"funciones. La notación regular de atributo por punto se usa para obtener y "
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Creo que este cambio no iría.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No estoy muy seguro de la razón detrás de los espacios dobles que hay en algunos textos, pero es por eso que lo agregué cuando lo vi por casualidad y noté que no lo tenía. Si no va, dame oportunidad de regresarlo como estaba y actualizo el commit en un rato más.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En inglés se usa el doble espacio después del punto seguido como regla. Yo cuando los encuentro, los borro, pero si los quieres dejar como están, me parece bien también 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gracias por el dato.

@migrmrz
Copy link
Contributor Author

migrmrz commented Dec 10, 2020

Hice un force para tener las correcciones en el mismo commit original, porque se me pasó el amend en los anteriores.

@cmaureir
Copy link
Collaborator

Hice un force para tener las correcciones en el mismo commit original, porque se me pasó el amend en los anteriores.

ten cuidado igual con eso, no recomendamos el force push justamente para ver la historia de todos los cambios que vamos haciendo, y la discusión con los comentarios. Al final hacemos un Squash and merge, asi que no te preocupes de llenar de commits el PR :)

@migrmrz
Copy link
Contributor Author

migrmrz commented Dec 11, 2020

ah, perfecto. Gracias.

@migrmrz
Copy link
Contributor Author

migrmrz commented Dec 18, 2020

Hola. Si esta parte está lista, ¿podríamos hacer merge? Quise tener mi fork actualizado pero no lo logré con fetch-checkout-merge desde mi terminal y terminé optando por borrarlo, pero olvidé que este PR seguía abierto y ahora mi siguiente commit abrirá uno nuevo :(

@cmaureir
Copy link
Collaborator

@migrmrz no te preocupes por abrir uno nuevo, podemos cerrar este y dejar el otro con estos cambios.

@migrmrz
Copy link
Contributor Author

migrmrz commented Dec 19, 2020

Entonces abro el nuevo PR y lo menciono para aclarar y cerrar éste. Ya luego me confirman si ustedes lo cierran o lo hago yo. Gracias

@cmaureir
Copy link
Collaborator

Entonces abro el nuevo PR y lo menciono para aclarar y cerrar éste. Ya luego me confirman si ustedes lo cierran o lo hago yo. Gracias

claro. Si lo puedes cerrar, adelante, si no, me dices y lo cierro.

migrmrz added a commit to migrmrz/python-docs-es that referenced this pull request Dec 22, 2020
Closes python#304. Se cierra PR python#1168 y se incluyen modificaciones conjuntas. Avance a 45%
@migrmrz migrmrz closed this Dec 22, 2020
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 reference/datamodel.po
3 participants