◐ Shell
reader mode source ↗
Skip to content

WeakMap and WeakSet#162

Merged
tarasyyyk merged 17 commits into
javascript-tutorial:masterfrom
MykolaSopiha:weakmap-weakset
Aug 3, 2021
Merged

WeakMap and WeakSet#162
tarasyyyk merged 17 commits into
javascript-tutorial:masterfrom
MykolaSopiha:weakmap-weakset

Conversation

@MykolaSopiha

@MykolaSopiha MykolaSopiha commented Jul 28, 2021 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Під час перекладу помітив, що інколи назви методів перелічуються в тексті з дужками, а іноді ні. Навіть в одному і тому ж реченні. Можливо це слід обговорити та прийти до єдиного стандарту. Поки що перекладав згідно з оригіналом:
image

@javascript-translate-bot javascript-translate-bot mentioned this pull request Jul 28, 2021
Open
tarasyyyk
tarasyyyk requested changes Jul 29, 2021
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

Дуже добре. Є декілька невеличких правок. Я вже додав пропозиції, тож залишилося лише їх закомітити. На рахунок назв методів. Про це не переймайтеся – пізніше в англійському варіанті їх заберуть і я оновлю переклад.

@javascript-translate-bot

javascript-translate-bot commented Jul 29, 2021

Copy link
Copy Markdown
Contributor

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

MykolaSopiha and others added 9 commits July 29, 2021 22:59
@MykolaSopiha
weakmap-weakset
eb1c521
@MykolaSopiha @tarasyyyk
Update 1-js/05-data-types/08-weakmap-weakset/article.md …
11577f6 
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha @tarasyyyk
Update 1-js/05-data-types/08-weakmap-weakset/article.md …
fb97f49 
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha @tarasyyyk
Update 1-js/05-data-types/08-weakmap-weakset/article.md …
2d442b9 
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha @tarasyyyk
Update 1-js/05-data-types/08-weakmap-weakset/article.md …
b57764a 
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha @tarasyyyk
Update 1-js/05-data-types/08-weakmap-weakset/article.md …
419af44 
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha @tarasyyyk
Update 1-js/05-data-types/08-weakmap-weakset/article.md …
8dac342 
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha @tarasyyyk
Update 1-js/05-data-types/08-weakmap-weakset/article.md …
f5eb970 
Co-authored-by: Taras <oster.ok@gmail.com>
@MykolaSopiha
Merge branch 'weakmap-weakset' of github.com:MykolaSopiha/uk.javascri… …
44c5f94 
…pt.info into weakmap-weakset
@MykolaSopiha

MykolaSopiha commented Jul 29, 2021

Copy link
Copy Markdown
Collaborator Author

@tarasyyyk дякую за правки! Я їх закомітив + додав ще пару невеликих фіксів. Сподіваюсь тепер все ок)

@MykolaSopiha

MykolaSopiha commented Aug 1, 2021

Copy link
Copy Markdown
Collaborator Author

/done

@tarasyyyk

tarasyyyk commented Aug 3, 2021

Copy link
Copy Markdown
Collaborator

@MykolaSopiha чудово! Вибачте за затримку. Зараз в мене обмаль часу на перевірки.
Переклад підтверджую.

@tarasyyyk tarasyyyk merged commit 89ee8e9 into javascript-tutorial:master Aug 3, 2021
@javascript-translate-bot

javascript-translate-bot commented Aug 3, 2021

Copy link
Copy Markdown
Contributor

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

Reviewers

@tarasyyyk tarasyyyk tarasyyyk approved these changes

Assignees

No one assigned

Labels

needs +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MykolaSopiha @javascript-translate-bot @tarasyyyk