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

Iterables #118

Open
wants to merge 28 commits into
base: master
Choose a base branch
from
Open

Iterables #118

wants to merge 28 commits into from

Conversation

marcoarib
Copy link

No description provided.

@CLAassistant
Copy link

CLAassistant commented Apr 27, 2020

CLA assistant check
All committers have signed the CLA.

@wnleao
Copy link
Contributor

wnleao commented May 9, 2020

Hey, Marco! Could you join all commits in only 1?

@wnleao
Copy link
Contributor

wnleao commented May 9, 2020

Btw, did you check if the english version had changes? Some pt-br articles are way back... https://github.com/javascript-tutorial/en.javascript.info

@marcoarib
Copy link
Author

Hey, Marco! Could you join all commits in only 1?

How can I do that? I'm not familiarized with Github editing.
Thanks.

@wnleao
Copy link
Contributor

wnleao commented May 9, 2020

Hey, Marco! Could you join all commits in only 1?

How can I do that? I'm not familiarized with Github editing.
Thanks.

No problem. Are you using the command line? Check out this link: https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git

@wnleao
Copy link
Contributor

wnleao commented May 9, 2020

Also, I've seen in your translation that you're translating code aswell. We should keep the source code as in the original and only translate comments.

@wnleao
Copy link
Contributor

wnleao commented May 9, 2020

Later on, whenever you want to change your last commit, like picking up from where you left off, you should use git commit --amend check out more details in https://www.atlassian.com/br/git/tutorials/rewriting-history

@marcoarib
Copy link
Author

Hey, Marco! Could you join all commits in only 1?

How can I do that? I'm not familiarized with Github editing.
Thanks.

No problem. Are you using the command line? Check out this link: https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git

I'm using browser.

@wnleao
Copy link
Contributor

wnleao commented May 10, 2020

Hey, marco! Could you finish up reviewing #121 ? I've already done a thorough review and Ana already changed what had to be changed. We need 2 people to review n order to approve the PR. Cheers!

@wnleao
Copy link
Contributor

wnleao commented May 11, 2020

Hey, Marco! Could you join all commits in only 1?

How can I do that? I'm not familiarized with Github editing.
Thanks.

No problem. Are you using the command line? Check out this link: https://stackoverflow.com/questions/5189560/squash-my-last-x-commits-together-using-git

I'm using browser.

I don't know how to squash commits using github site, but let me know when you're done and I'll try to squash your commits over here. Ok?

@odsantos
Copy link
Contributor

odsantos commented Dec 6, 2020

Hi @marcoarib please could you also resolve the conflicts? Thank you.

@odsantos
Copy link
Contributor

Hi @marcoarib , "Object.keys, values, entries" is not part of "Iterables".
Would you remove it from this Pull Request and create a separate one for it?
Would you also resolve the conflicts?
Thanks.

@nazarepiedady
Copy link
Member

@Peruibeloko, @jonnathan-ls, @odsantos, we need some solution to this pull request. I am thinking on to close it and start a new one with updates applied and conflicts resolved.

@Peruibeloko
Copy link
Contributor

@nazarepiedady I'll fork and update it

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

Successfully merging this pull request may close these issues.

7 participants