Skip to content

Sincronización 5 Julio #293

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

Merged
merged 8 commits into from
Jul 9, 2020
Merged

Sincronización 5 Julio #293

merged 8 commits into from
Jul 9, 2020

Conversation

vplentinax
Copy link
Contributor

Repo actualizada. Añadidos cambios de la repo EN desde los últimos 25 días después de la última actualización. Si el ariculo estaba traducido, añadí el cambio traducido.

@javascript-translate-bot javascript-translate-bot requested a review from a team July 5, 2020 20:16
@vplentinax vplentinax changed the title Sincronización 5 July Jul 6, 2020
Copy link
Contributor

@EzequielCaste EzequielCaste left a comment

Choose a reason for hiding this comment

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

Ok con algunos cambios

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

vplentinax and others added 5 commits July 6, 2020 14:39
Co-authored-by: Ezequiel Castellanos <51804994+ezzep66@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+ezzep66@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+ezzep66@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+ezzep66@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+ezzep66@users.noreply.github.com>
@vplentinax
Copy link
Contributor Author

/done

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

No problem. Un solo cambio, hagan commit de eso y consideren que lo aprobé.

Hay cambios en ingles que afectan PR en curso:
no importa, serán conflictos simples. Y no importa quien merge primero.

Hay cambios que afectan ya traducidos:

1- 6- 6 / 5 agregas algo al ejercicio, no probe como lo afecta, asumo que si hubiera que cambiarlo lo habrias visto.

Problema de ejercicios agregados: Cuando article ya está traducido.
Cuando hagas merge hay que destildarlo para traducir el ejercicio:
1-9-3-3
[X] Static properties and methods (@cortizg) #172

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax
Copy link
Contributor Author

/done

@vplentinax
Copy link
Contributor Author

Confirmen... @joaquinelio @ezzep66

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

@vplentinax , Ok.
cc @ezzep66
solo 12 PR, de los 80 al empezar y el repo abandonado,
y el más viejo de 2 semanas...
más: de los pocos y mal contabilizados traducidos estamos a menos de diez de la mitad del repo.
Comming soon:
Para el peor lenguaje al que estamos condenados: El MEJOR tutorial de la web.
🍺 🍰 y deberia estar c4 en la fiesta.

Bueno.
¿Será tiempo de estudiar los pr foráneos...? en lugar del artesanal.
Creo que no rompen nada.
Anduve perezoso, pero se puede probar con el pr sobre un branch local,
serán MUCHOS conflictos, pero no debieran ser más que los que estás haciendo manualmente. Si son más es porque los estamos perdiendo.
Lo único que desanima es el terrible sistema del git para resolverlos, agergando marcas y las dos versiones en el arch de trabajo. ¿Habrá otra forma?
Además es la manera correcta. Al sincronizar sin pull se pierde su registro de commits (y colaboradores de inglés), no me opuse porque esto no es código.
Veremos...

@EzequielCaste EzequielCaste merged commit c2a0dc3 into javascript-tutorial:master Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 participants