Skip to content

Loops: while and for#84

Merged
bugron merged 31 commits into
javascript-tutorial:masterfrom
vahmelk99:master
Aug 21, 2021
Merged

Loops: while and for#84
bugron merged 31 commits into
javascript-tutorial:masterfrom
vahmelk99:master

Conversation

@vahmelk99

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated

@bugron bugron left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overall looks good just a few comments to address. Besides those comments while you're resolving merge conflicts I'll get to reviewing the main article.

Comment thread 1-js/02-first-steps/13-while-for/2-which-value-while/solution.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/5-replace-for-while/task.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/6-repeat-until-correct/solution.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/7-list-primes/solution.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/7-list-primes/solution.md
Comment thread 1-js/02-first-steps/13-while-for/7-list-primes/task.md Outdated
@javascript-translate-bot

Copy link
Copy Markdown

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

@vahmelk99

Copy link
Copy Markdown
Contributor Author

Thanks for the review)

@vahmelk99

Copy link
Copy Markdown
Contributor Author

/done

@bugron bugron left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's address a few more comments and we're good to go.

Comment thread 1-js/02-first-steps/13-while-for/7-list-primes/solution.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
Comment thread 1-js/02-first-steps/13-while-for/article.md Outdated
@javascript-translate-bot

Copy link
Copy Markdown

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

vahmelk99 and others added 18 commits August 21, 2021 19:31
@vahmelk99

Copy link
Copy Markdown
Contributor Author

/done

@bugron

bugron commented Aug 21, 2021

Copy link
Copy Markdown
Contributor

This is awesome! Thanks @vahmelk99 for all your hard work. It is my pleasure to collaborate with you. Way to go! Merged.

@bugron bugron merged commit dd52f90 into javascript-tutorial:master Aug 21, 2021
@javascript-translate-bot

Copy link
Copy Markdown

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

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.

3 participants