Skip to content

Modules, introduction - #198

Merged
EzequielCaste merged 2 commits into
javascript-tutorial:masterfrom
EzequielCaste:modules_intro
May 27, 2020
Merged

Modules, introduction#198
EzequielCaste merged 2 commits into
javascript-tutorial:masterfrom
EzequielCaste:modules_intro

Conversation

@EzequielCaste

@EzequielCasteEzequielCaste commented May 19, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@javascript-translate-bot
javascript-translate-bot requested a review from a teamMay 19, 2020 22:52
@javascript-translate-bot

javascript-translate-bot commented May 19, 2020

Copy link
Copy Markdown

Error: the article already has PR number in the Progress Issue #17, it's 197 ⁉️

@iliakaniliakan mentioned this pull request May 19, 2020
Open
joaquinelio
joaquinelio approved these changes May 25, 2020
View reviewed changes

@joaquineliojoaquinelio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Me gusta.
Me gustarían algunos cambios pero es gusto personal o cosas que no hacen al JS...

y tenemos 70 y pico de PR por meter, el doble de review

Comment thread1-js/13-modules/01-modules-intro/article.md Outdated
Comment thread1-js/13-modules/01-modules-intro/article.md Outdated
Comment thread1-js/13-modules/01-modules-intro/article.md
@EzequielCaste
EzequielCaste merged commit 4ad4b7f into javascript-tutorial:masterMay 27, 2020
@javascript-translate-bot

javascript-translate-bot commented May 27, 2020

Copy link
Copy Markdown

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

@joaquinelio

joaquinelio commented May 27, 2020

Copy link
Copy Markdown
Member

@ezzep66 ¡Qué emoción!
log

Una cosita... El bot sugiere 2 reviewers (quedó colgado el status needs+1) , cmo maintainer es tu prerrogativa no hacerle caso ¿vamos a trabajar así?
Por mí BIEN, prefiero que se filtre algún errorcito a ver que diluye por falta de interés como el año pasado. Además cuando se termine el resto se pueden revisar los PR cerrados ver cuál necesita rechequeo, no me preocupa.

Era deprimente el mensaje "last commit" parado un año. Podemos decir que avanza,

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

3 participants

@EzequielCaste@javascript-translate-bot@joaquinelio