-
Notifications
You must be signed in to change notification settings - Fork 112
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
base: master
Are you sure you want to change the base?
Iterables #118
Conversation
Hey, Marco! Could you join all commits in only 1? |
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 |
How can I do that? I'm not familiarized with Github editing. |
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 |
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. |
Later on, whenever you want to change your last commit, like picking up from where you left off, you should use |
I'm using browser. |
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! |
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? |
Hi @marcoarib please could you also resolve the conflicts? Thank you. |
Hi @marcoarib , "Object.keys, values, entries" is not part of "Iterables". |
@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 I'll fork and update it |
No description provided.