pt.javascript.info icon indicating copy to clipboard operation
pt.javascript.info copied to clipboard

Iterables

Open marcoarib opened this issue 4 years ago • 14 comments

marcoarib avatar Apr 27 '20 01:04 marcoarib

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Apr 27 '20 01:04 CLAassistant

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

wnleao avatar May 09 '20 17:05 wnleao

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

wnleao avatar May 09 '20 17:05 wnleao

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

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

marcoarib avatar May 09 '20 22:05 marcoarib

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 avatar May 09 '20 23:05 wnleao

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 avatar May 09 '20 23:05 wnleao

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

wnleao avatar May 09 '20 23:05 wnleao

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.

marcoarib avatar May 10 '20 02:05 marcoarib

Hey, marco! Could you finish up reviewing https://github.com/javascript-tutorial/pt.javascript.info/pull/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 avatar May 10 '20 02:05 wnleao

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?

wnleao avatar May 11 '20 21:05 wnleao

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

odsantos avatar Dec 06 '20 19:12 odsantos

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.

odsantos avatar Jan 28 '23 19:01 odsantos

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

nazarepiedady avatar Mar 10 '24 05:03 nazarepiedady

@nazarepiedady I'll fork and update it

peruibeloko avatar Mar 23 '24 21:03 peruibeloko