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

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

Already on GitHub?Sign in to your account

Closed
migrmrz wants to merge1 commit intopython:3.8frommigrmrz:translate-datamodel
Closed

Traducción reference/datamodel.po#1168

migrmrz wants to merge1 commit intopython:3.8frommigrmrz:translate-datamodel

Conversation

migrmrz
Copy link
Contributor

@migrmrzmigrmrz commentedDec 9, 2020
edited by cmaureir
Loading

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
ContributorAuthor

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 👍

migrmrz reacted with thumbs up emoji
Copy link
ContributorAuthor

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
ContributorAuthor

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
ContributorAuthor

ah, perfecto. Gracias.

@migrmrz
Copy link
ContributorAuthor

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 reacted with thumbs up emoji

@migrmrz
Copy link
ContributorAuthor

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 reacted with rocket emoji

migrmrz added a commit to migrmrz/python-docs-es that referenced this pull requestDec 22, 2020
Closespython#304. Se cierra PRpython#1168 y se incluyen modificaciones conjuntas. Avance a 45%
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@cmaureircmaureircmaureir left review comments

@eamanueamanueamanu left review comments

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

Successfully merging this pull request may close these issues.

Translatereference/datamodel.po
3 participants
@migrmrz@cmaureir@eamanu

[8]ページ先頭

©2009-2025 Movatter.jp