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.

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

Currying #287

Merged
merged 15 commits into from Jul 18, 2020
Merged

Currying #287

merged 15 commits into from Jul 18, 2020

Conversation

teocicciari
Copy link
Contributor

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from a team June 29, 2020 02:06
@iliakan iliakan mentioned this pull request Jun 29, 2020
Open
@CLAassistant
Copy link

CLAassistant commented Jun 29, 2020 •
edited

CLA assistant check
All committers have signed the CLA.

joaquinelio
joaquinelio requested changes Jul 5, 2020
View reviewed changes
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.

Muy clara,
cambios menores.
Por favor, pedimos respetar la numeración de líneas original,
pues facilita tanto la revisión como la actualización con el repo inglés.
Creo que los suggest arreglan la numeración pero por favor probalo, puedo equivocarme.

1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/03-currying-partials/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot
Copy link

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

@joaquinelio
Copy link
Member

@vplentinax
este es simple, pero requiere maintainer SALVO que estés con algo gordo (como sueles).

No respondio en 2 semanas, pero no vi mal el articulo.
Podrias darle una leida, hacerle mis commit si estas dw acuerdo o los reviso si no,
y asi cerramos el circulo

vplentinax and others added 13 commits July 17, 2020 20:29
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
b310c66 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
e6434b3 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
319bd2b 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
c8dceaa 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
b0bd653 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
f7ab311 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
90a0498 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
a923c95 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
428b5ec 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
8404826 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
3915bd2 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
38fe5e8 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
f49de17 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax
Copy link
Contributor

/done

@vplentinax @joaquinelio
Update 1-js/99-js-misc/03-currying-partials/article.md
b8f7f2f 
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax
Copy link
Contributor

/done

@vplentinax
Copy link
Contributor

Se arregló el problema de las líneas! @joaquinelio

maksumi
maksumi approved these changes Jul 18, 2020
View reviewed changes
Copy link
Collaborator

@maksumi maksumi left a comment

Choose a reason for hiding this comment

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

Its ok!!!

vplentinax
vplentinax approved these changes Jul 18, 2020
View reviewed changes
Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

Nos fuimos!

@joaquinelio joaquinelio merged commit 6c54880 into javascript-tutorial:master Jul 18, 2020
@javascript-translate-bot
Copy link

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

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

@joaquinelio joaquinelio joaquinelio approved these changes

@maksumi maksumi maksumi approved these changes

@vplentinax vplentinax vplentinax approved these changes

Assignees
No one assigned
Labels
ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@teocicciari @CLAassistant @javascript-translate-bot @joaquinelio @vplentinax @maksumi