Skip to content

Working on inputoutput. #46

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 5, 2020
Merged

Working on inputoutput. #46

merged 2 commits into from
May 5, 2020

Conversation

kikocorreoso
Copy link
Contributor

improvements on inputoutput.mo

humitos
humitos approved these changes May 3, 2020
View reviewed changes
Copy link
Collaborator

@humitos humitos left a comment

Choose a reason for hiding this comment

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

👌

humitos
humitos reviewed May 3, 2020
View reviewed changes
Copy link
Collaborator

@humitos humitos left a comment

Choose a reason for hiding this comment

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

.

tutorial/inputoutput.po
@@ -290,6 +290,8 @@ msgid ""
"(Note that the one space between each column was added by the way :func:"
"`print` works: it always adds spaces between its arguments.)"
msgstr ""
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esta traducción me gusta más que la de la otra pr abierta #45

Copy link
Contributor

Choose a reason for hiding this comment

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

+1

@kikocorreoso
Copy link
Contributor Author

kikocorreoso commented May 4, 2020 •
edited
Loading

Mis disculpas, no me di cuenta de la traducción de @eamanu

Quizá se podrían abrir issues con lo que queda por traducir, un issue por cada fichero po y que cada cual se asigne el po que considere y esté libre.

También, reflejar este flujo de trabajo en el README para que cualquiera sepa como hacerlo.

@humitos
Copy link
Collaborator

humitos commented May 4, 2020

Quizá se podrían abrir issues con lo que queda por traducir, un issue por cada fichero po y que cada cual se asigne el po que considere y esté libre.

Sí. Me gusta la idea. Tenemos más colaboradores de los que esperábamos :) --y ya empezamos a notar que necesitamos organizarnos mejor.

También, reflejar este flujo de trabajo en el README para que cualquiera sepa como hacerlo.

El README necesita bastante trabajo todavía:

El CONTRIBUTING debería mencionar esto que dices.

Mi idea es tener muchos puntos de entrada (GitHub, lista de correo, telegram, lo que sea) pero que todos apunten a la misma página de contributing. Sino, mantener toda esta info en diferentes lugares va a ser una locura.

Creo que podemos hacer páginas lindas con RST (como la de contributing) y manejar todo esto desde ahí.

@humitos
Copy link
Collaborator

humitos commented May 4, 2020

@kikocorreoso ¿te gustaría mejorar eso en esa dirección?

humitos
humitos approved these changes May 4, 2020
View reviewed changes
Copy link
Collaborator

@humitos humitos left a comment

Choose a reason for hiding this comment

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

El PR está genial. Falta arreglar los conflictos y listo. Hay traducciones que están mejor en esta PR que en la que genera conflicto, así que algunas hay que pisarlas con estas.

@kikocorreoso
Copy link
Contributor Author

El PR está genial. Falta arreglar los conflictos y listo. Hay traducciones que están mejor en esta PR que en la que genera conflicto, así que algunas hay que pisarlas con estas.

@humitos En lo de arreglar conflictos, ¿es algo que tengo que hacer yo o el que mergea?

@kikocorreoso ¿te gustaría mejorar eso en esa dirección?

@humitos ¿con lo anterior te refieres a si quiero arreglar el README y dedicarme a hacer triaje para crear issues individuales con cosas en las que otros podrían colaborar?

@humitos
Copy link
Collaborator

humitos commented May 4, 2020

@humitos En lo de arreglar conflictos, ¿es algo que tengo que hacer yo o el que mergea?

Si lo hace el que propone la PR, por mí, mejor. Así queda lista la PR para mergear y solo hacemos click en "Merge pull request".

@humitos ¿con lo anterior te refieres a si quiero arreglar el README y dedicarme a hacer triaje para crear issues individuales con cosas en las que otros podrían colaborar?

Arreglar el README y explicar el proceso de asignación de issues en CONTRIBUTING. El "triage" supongo que se hará más o menos solo, ¿no?. Cada uno busca un issue y se lo asigna o escribe un comentario si no tiene permisos. Igual, movamos esta charla sobre el proceso a #52.

@gilgamezh
Copy link
Collaborator

@kikocorreoso podrias por favor hacer fix de los conflictos? creo que con eso ya estamos listos para mergear

@gilgamezh gilgamezh self-requested a review May 4, 2020 19:09
gilgamezh
gilgamezh previously requested changes May 4, 2020
View reviewed changes
Copy link
Collaborator

@gilgamezh gilgamezh left a comment

Choose a reason for hiding this comment

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

Resolver conflictos por favor

@humitos humitos requested a review from gilgamezh May 5, 2020 09:47
@humitos humitos dismissed gilgamezh’s stale review May 5, 2020 09:47

Ya lo hizo :)

@humitos humitos merged commit d206dcd into python:3.7 May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eamanu eamanu eamanu left review comments

@humitos humitos humitos approved these changes

@raulcd raulcd raulcd approved these changes

@gilgamezh gilgamezh Awaiting requested review from gilgamezh

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

Successfully merging this pull request may close these issues.
5 participants
@kikocorreoso @humitos @gilgamezh @raulcd @eamanu