Skip to content

Numbers - #62

Merged
tarasyyyk merged 6 commits into
javascript-tutorial:masterfrom
alexandrtovmach:translation/js-dataTypes-number
Mar 6, 2020
Merged

Numbers#62
tarasyyyk merged 6 commits into
javascript-tutorial:masterfrom
alexandrtovmach:translation/js-dataTypes-number

Conversation

@alexandrtovmach

@alexandrtovmachalexandrtovmach commented Mar 5, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@javascript-translate-bot
javascript-translate-bot requested a review from a teamMarch 5, 2020 09:32
@CLAassistant

CLAassistant commented Mar 5, 2020•
edited
Loading

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

tarasyyyk
tarasyyyk reviewed Mar 5, 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.

Very correct translation! No typos at all.

Comment thread1-js/05-data-types/02-number/article.md Outdated
Comment thread1-js/05-data-types/02-number/article.md Outdated
@alexandrtovmach@tarasyyyk
Apply suggestions from code review
92fc77f
Co-Authored-By: Taras <oster.ok@gmail.com>
alexgalkin
alexgalkin reviewed Mar 6, 2020
View reviewed changes
Comment thread1-js/05-data-types/02-number/article.md Outdated
alexgalkin
alexgalkin reviewed Mar 6, 2020
View reviewed changes

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

Чудовий початок!

@alexandrtovmach
alexandrtovmach marked this pull request as ready for review March 6, 2020 10:41
@alexandrtovmach

alexandrtovmach commented Mar 6, 2020

Copy link
Copy Markdown
ContributorAuthor

@tarasyyyk@alexgalkin
переклад завершено, можна ревьювити та мерджити

tarasyyyk
tarasyyyk requested changes Mar 6, 2020
View reviewed changes

@tarasyyyktarasyyyk left a comment •
edited
Loading

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.

Впринципі, немає помилок в перекладі. Можна мерджити, але є деякі зауваження.
@alexgalkin гляньте «незамилиним» оком на мої виправлення, чи згідні ви з ними.

Comment thread1-js/05-data-types/02-number/1-sum-interface/solution.md Outdated
Comment thread1-js/05-data-types/02-number/2-why-rounded-down/solution.md
Comment thread1-js/05-data-types/02-number/2-why-rounded-down/solution.md Outdated
Comment thread1-js/05-data-types/02-number/article.md Outdated
@javascript-translate-bot

javascript-translate-bot commented Mar 6, 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 Mar 6, 2020
Open
@alexandrtovmach@tarasyyyk
Apply suggestions from code review
49766ed
Co-Authored-By: Taras <oster.ok@gmail.com>
@alexandrtovmach

alexandrtovmach commented Mar 6, 2020

Copy link
Copy Markdown
ContributorAuthor

/done

@javascript-translate-bot
javascript-translate-bot requested a review from a teamMarch 6, 2020 19:47
@tarasyyyk
tarasyyyk merged commit 796b2b4 into javascript-tutorial:masterMar 6, 2020
@alexandrtovmach
alexandrtovmach deleted the translation/js-dataTypes-number branch March 6, 2020 20:45
@tarasyyyk

tarasyyyk commented Mar 6, 2020

Copy link
Copy Markdown
Collaborator

Thanks for your contribution! @all-contributors please add @alexandrtovmach to list for translation.

@allcontributorsallcontributorsBot mentioned this pull request Mar 6, 2020
Merged
@allcontributors

allcontributorsBot commented Mar 6, 2020

Copy link
Copy Markdown
Contributor

@tarasyyyk

I've put up a pull request to add @alexandrtovmach! 🎉

@alexgalkin

alexgalkin commented Mar 6, 2020

Copy link
Copy Markdown
Contributor

@alexandrtovmach amazing example of translation. Thanks for your contribution!

@tarasyyyktarasyyyk changed the title Translation for js-dataTypes-numberNumbersAug 29, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tarasyyyktarasyyyktarasyyyk approved these changes
+1 more reviewer
@alexgalkinalexgalkinalexgalkin left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@alexandrtovmach@CLAassistant@javascript-translate-bot@tarasyyyk@alexgalkin