Skip to content

Comments - #60

Merged
tarasyyyk merged 14 commits into
javascript-tutorial:masterfrom
alexgalkin:translation/comments
Feb 29, 2020
Merged

Comments#60
tarasyyyk merged 14 commits into
javascript-tutorial:masterfrom
alexgalkin:translation/comments

Conversation

@alexgalkin

@alexgalkinalexgalkin commented Feb 28, 2020

Copy link
Copy Markdown
Contributor

Translation of the "Comments" section.

tarasyyyk
tarasyyyk requested changes Feb 29, 2020
View reviewed changes

@tarasyyyktarasyyyk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Small corrections

Comment thread1-js/03-code-quality/03-comments/article.md Outdated
Comment thread1-js/03-code-quality/03-comments/article.md Outdated
Comment thread1-js/03-code-quality/03-comments/article.md Outdated

*!*
// check if i is a prime number
// перевірка чи є i простим числом

@tarasyyyktarasyyykFeb 29, 2020

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// перевірка чи є i простим числом
// перевірка чи є 'i' простим числом

Comment thread1-js/03-code-quality/03-comments/article.md Outdated
Comment thread1-js/03-code-quality/03-comments/article.md Outdated
Comment thread1-js/03-code-quality/03-comments/article.md
}
```
Наприклад:

@tarasyyyktarasyyykFeb 29, 2020

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Remove extra space.
It's better to add this one to English version as PR and this will be merged into Ukrainian if approved.

@alexgalkinalexgalkinFeb 29, 2020

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Created separate PR javascript-tutorial/en.javascript.info#1780 for this in English version

Comment thread1-js/03-code-quality/03-comments/article.md Outdated
Comment thread1-js/03-code-quality/03-comments/article.md Outdated
Comment thread1-js/03-code-quality/03-comments/article.md Outdated
@javascript-translate-bot

javascript-translate-bot commented Feb 29, 2020

Copy link
Copy Markdown
Contributor

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

@iliakaniliakan mentioned this pull request Feb 29, 2020
Open
alexgalkinand others added 8 commits February 29, 2020 22:48
@alexgalkin@tarasyyyk
Update 1-js/03-code-quality/03-comments/article.md
8ef3e3a
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexgalkin@tarasyyyk
Update 1-js/03-code-quality/03-comments/article.md
49c4109
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexgalkin@tarasyyyk
Update 1-js/03-code-quality/03-comments/article.md
0f5d422
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexgalkin@tarasyyyk
Update 1-js/03-code-quality/03-comments/article.md
379d44a
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexgalkin@tarasyyyk
Update 1-js/03-code-quality/03-comments/article.md
6630148
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexgalkin@tarasyyyk
Update 1-js/03-code-quality/03-comments/article.md
bcdc3d1
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexgalkin@tarasyyyk
Update 1-js/03-code-quality/03-comments/article.md
68ca202
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexgalkin
Small correction to variable i ref
e731e5a
@alexgalkin

alexgalkin commented Feb 29, 2020

Copy link
Copy Markdown
ContributorAuthor

/done

@alexgalkin
alexgalkin requested review from tarasyyyk and removed request for a teamFebruary 29, 2020 21:00
@tarasyyyk
tarasyyyk merged commit aabd08b into javascript-tutorial:masterFeb 29, 2020
@tarasyyyktarasyyyk changed the title Translation of "Comments" sectionCommentsAug 29, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tarasyyyktarasyyykAwaiting requested review from tarasyyyk

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexgalkin@javascript-translate-bot@tarasyyyk