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 reference/introduction#2716

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
mmmarcos merged 27 commits intopython:3.12fromJsMelix:traduccion-introduction.po
Jan 27, 2024

Conversation

JsMelix
Copy link
Contributor

@JsMelixJsMelix commentedOct 28, 2023
edited
Loading

Closes#2646

@rtobar
Copy link
Collaborator

@JsMelix gracias por la contribución. Podrías modificar la descripción de este PR para que diga "Closes #nnnn" como de explica en el paso 8 dehttps://python-docs-es.readthedocs.io/es/3.12/CONTRIBUTING.html#paso-2-comienza-a-traducir? Así quedea todo linkeado con el issue original, el ual se cierra automáticamente cuando esté PR se mergee. Gracias!

JsMelix reacted with thumbs up emoji

@cmaureir
Copy link
Collaborator

¿Por qué lo cerraste@JsMelix ?

@JsMelixJsMelix reopened thisNov 1, 2023
@JsMelix
Copy link
ContributorAuthor

sorry, :) ya esta

Copy link
ContributorAuthor

@JsMelixJsMelix left a comment

Choose a reason for hiding this comment

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

Closes#2716

@rtobar
Copy link
Collaborator

@JsMelix gracias por actualizar la descripción del PR. Sin embargo ahora hay tests en CI que aún están fallando. Fíjate en los mensajes de error, y por favor corrige lo que sea necesario, así haces la tarea de revisión más fácil. Puedes ver enhttps://python-docs-es.readthedocs.io/es/3.12/reviewers-guide.html algunos errores comunes de los distintos chequeos que se realizan.

Copy link
Collaborator

@rtobarrtobar left a comment

Choose a reason for hiding this comment

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

Ahí van algunos comentarios, gracias por la contribución!

@rtobar
Copy link
Collaborator

@JsMelix no estoy seguro, pero después de leer tus respuestas tengo la impresión (quizás incorrecta) de que tú crees que mis comentarios son más que nada informativos, y que no falta nada que hacer. Desafortunadamente el proceso no ha terminado, y la idea es que vayas incorporando los comentarios que te la gente va dejando como parte de la revisión. Cuando ya creamos que está todo bien entonces marcaremos el PR como "listo" y lo mergeamos 👍.

Como dices tú, se aprende de los errores, pero dentro de esta misma contribución 😄

@JsMelix
Copy link
ContributorAuthor

Closes#2646

@github-actionsGitHub Actions
Copy link

All entries translated, horray! 🎉

@rtobarrtobar mentioned this pull requestDec 4, 2023
@JsMelixJsMelix closed thisDec 4, 2023
@JsMelixJsMelix reopened thisDec 4, 2023
Copy link
Collaborator

@mmmarcosmmmarcos left a comment

Choose a reason for hiding this comment

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

@JsMelix gracias por la contribución. Y gracias@rtobar y@cacrespo por los reviews.
Marqué como resolved los request changes que estaban abiertos y fueron tratados.
Me parece que ahora está todo bien para mergear 👍

JsMelix reacted with hooray emoji
Copy link
Collaborator

@mmmarcosmmmarcos left a comment

Choose a reason for hiding this comment

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

Ooops quedaron algunos términos que no pasan el cspell.

@JsMelix Tendrías que agregar dentro del directoriodictionaries un archivo llamadoreference_introduction.txt con el siguiente contenido:

choseclonadoraEnglishstackless

JsMelix reacted with thumbs up emoji
@JsMelix
Copy link
ContributorAuthor

Ya realicé los cambios mencionados :D

mmmarcos reacted with thumbs up emoji

Copy link
Collaborator

@mmmarcosmmmarcos left a comment

Choose a reason for hiding this comment

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

Ahora sí, pasaron los checks 🙌

JsMelix reacted with hooray emoji
@mmmarcosmmmarcos merged commita752393 intopython:3.12Jan 27, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@rtobarrtobarrtobar left review comments

@cacrespocacrespocacrespo left review comments

@mmmarcosmmmarcosmmmarcos 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.

Translatereference/introduction.po
6 participants
@JsMelix@rtobar@cmaureir@mmmarcos@cacrespo@Sebastianxeii

[8]ページ先頭

©2009-2025 Movatter.jp