◐ Shell
reader mode source ↗
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

@alexandrtovmach alexandrtovmach commented Mar 5, 2020

Copy link
Copy Markdown
Contributor

No description provided.

@javascript-translate-bot javascript-translate-bot requested a review from a team March 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

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

Hide comment

Very correct translation! No typos at all.

@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

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

Hide comment

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

@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
Contributor Author

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

tarasyyyk
tarasyyyk requested changes Mar 6, 2020
View reviewed changes

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

Hide comment

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

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

@iliakan iliakan 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
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team March 6, 2020 19:47
@tarasyyyk tarasyyyk merged commit 796b2b4 into javascript-tutorial:master Mar 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.

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

allcontributors Bot 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!

@tarasyyyk tarasyyyk removed the needs +1 label Mar 6, 2020
@tarasyyyk tarasyyyk changed the title Translation for js-dataTypes-number Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tarasyyyk tarasyyyk

+1 more reviewer

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