Skip to content
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

Map and Set #263

Merged
merged 24 commits into from Jul 5, 2020
Merged

Map and Set #263

merged 24 commits into from Jul 5, 2020

Conversation

@vplentinax
Copy link
Contributor

@vplentinax vplentinax commented Jun 18, 2020

Arreglado los errores de formato.

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jun 18, 2020
@iliakan iliakan mentioned this pull request Jun 18, 2020
159 of 174 tasks complete
@EzequielCaste EzequielCaste self-requested a review Jul 4, 2020
Copy link
Contributor

@EzequielCaste EzequielCaste left a comment

Hay error al comienzo de 1-js/05-data-types/07-map-set/02-filter-anagrams/solution.md
No se ha respetado los números de las líneas. Debe comenzar en línea N° 1 y no en la N°2.

Por favor corregir así puedo seguir con la revisión.

@javascript-translate-bot

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

arreglé el problema de los espacios.
Por favor tienen que coincidir los números de las líneas con el documento Original.
problemas con la numeración de líneas
número de líneas
respetar el número de línea
Copy link
Contributor

@EzequielCaste EzequielCaste left a comment

muchos problemas con espacios donde no deberían estar.
Se debe respetar el número de línea del documento original.

1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
1-js/05-data-types/07-map-set/article.md Outdated Show resolved Hide resolved
@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 12 commits Jul 4, 2020
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>
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>
Co-authored-by: Ezequiel Castellanos <51804994+ezzep66@users.noreply.github.com>
Co-authored-by: Ezequiel Castellanos <51804994+ezzep66@users.noreply.github.com>
vplentinax and others added 5 commits Jul 4, 2020
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

@vplentinax vplentinax commented Jul 4, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jul 4, 2020
@vplentinax vplentinax merged commit 6249f2a into javascript-tutorial:master Jul 5, 2020
1 check passed
1 check passed
license/cla Contributor License Agreement is signed.
Details
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

@joaquinelio
Copy link
Contributor

@joaquinelio joaquinelio commented Jul 5, 2020

@vplentinax @ezzep66
¿Merged? Lo tenía recién en una pestaña para revisión...
Estoy de ACUERDO con los merge aprobados por la trusted people, (después de la carga, revisar el repo entero)
¿pero qué pasó con eso del doble review que pide don Bot?
No me quejo, es saber cuál es el criterio.

@vplentinax
Copy link
Contributor Author

@vplentinax vplentinax commented Jul 5, 2020

En mis PR el segundo review lo doy yo jejeje.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.