Skip to content

Map and Set#263

Merged
vplentinax merged 24 commits into
javascript-tutorial:masterfrom
vplentinax:mset
Jul 5, 2020
Merged

Map and Set#263
vplentinax merged 24 commits into
javascript-tutorial:masterfrom
vplentinax:mset

Conversation

@vplentinax

Copy link
Copy Markdown
Contributor

Arreglado los errores de formato.

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md
Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md Outdated
Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md
Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md Outdated
Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md
Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md
Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md
Comment thread 1-js/05-data-types/07-map-set/01-array-unique-map/task.md Outdated
@javascript-translate-bot

Copy link
Copy Markdown

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

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

Comment thread 1-js/05-data-types/07-map-set/03-iterable-keys/task.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
Comment thread 1-js/05-data-types/07-map-set/article.md Outdated
@javascript-translate-bot

Copy link
Copy Markdown

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 July 4, 2020 19:15
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 July 4, 2020 19:18
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
Copy Markdown
Contributor Author

/done

@vplentinax vplentinax merged commit 6249f2a into javascript-tutorial:master Jul 5, 2020
@javascript-translate-bot

Copy link
Copy Markdown

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

@joaquinelio

joaquinelio commented Jul 5, 2020

Copy link
Copy Markdown
Member

@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
Copy Markdown
Contributor Author

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants