Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Добавление раздела: WeakRef & FinalizationRegistry #1899

Merged
merged 6 commits into from
Oct 31, 2023

Conversation

WOLFRIEND
Copy link
Contributor

Описание

Добавление информации о таких возможностях как WeakRef & FinalizationRegistry.
Обсуждения и утверждение данного предложения находятся по ссылке:
#1898

Ссылки

Связанные Issue

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Oct 12, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team October 12, 2023 10:16
@CLAassistant
Copy link

CLAassistant commented Oct 12, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@Alexandre887 Alexandre887 left a comment

Choose a reason for hiding this comment

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

Добрый вечер.

Не пугайтесь большого количества правок — в основном это suggested changes, которые можно принять в один клик.

Также:

  • Я оставил комментарии к некоторым строкам кода, т.к не везде понятна ваша мысль.
  • Я оставил комментарии к некоторым частям текста, где есть тавтология и/или логические ошибки. Также где-то определённые выражения звучат не совсем по-русски, так что их нужно перефразировать.
  • Были ошибки, которые распространяются на весь материал статьи. К примеру, неверное написание определённых слов. В таком случае я не искал и не исправлял эти ошибки во всём тексте, просто оставил комментарий к одному подобному примеру, дальше вы должны будете сами их исправить.

Обязательно сначала примите suggested changes, а только потом рассматривайте комментарии. Иногда я оставлял и suggested changes, и комментарий в одной правке, так что внимательно всё прочтите.

Если мои замечания где-то не совсем понятны, или вы хотите их оспорить, — в Github также можно оставлять к ним комментарии, я получу уведомление и мы обсудим эти вопросы.

В техническую часть материала я пока глубоко не вдавался, сфокусировался на тексте и коде.

В раздел «Типы данных», на мой взгляд, статью будет помещать неправильно. Это очень узконаправленная тема, которая на практике не затрагивает 90% всех программистов. Многие спокойно пишут код, даже не зная, что такие возможности были введены в язык. Оно и к лучшему, т.к даже в mdn приведён перечень причин, по которым лучше избегать использования структур, о которых идет речь в статье.
Поэтому, лучше всего будет добавить статью в раздел «Разное», в конец первой части учебника.

В начале статьи можем добавить предостережение, указать в нём причины, которые я привёл выше. Также следует добавить информацию о том, что для прочтения необходимы базовые знания в DOM, т.к в статье есть примеры с его использованием.
Но, сейчас это не столь важно. Сосредоточьтесь на рассмотрении моих правок.

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Oct 22, 2023
@javascript-translate-bot
Copy link
Contributor

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

@WOLFRIEND
Copy link
Contributor Author

Спасибо большое за правки и предложение.
Я ознакомился и принял ваши suggested changes.
Теперь начинаю работу над вашими комментариями.

@WOLFRIEND
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Oct 25, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team October 25, 2023 08:30
@WOLFRIEND
Copy link
Contributor Author

Добрый день,

Я внёс изменения в соответствии с вашими правками:

  1. Принял ваши рекомендации.
  2. Исправил перевод текста в необходимых местах (картинки, примеры).
  3. Устранил ошибки, исправил непонятные описания и объяснения в соответствии с вашими комментариями.
  4. Перенёс статью в раздел, который вы предложили.
  5. Добавил необходимые предупреждения.

Я старался максимально следовать вашим правкам и комментариям. Пожалуйста, оцените результат и скажите, нужны ли дополнительные правки к материалу? Спасибо за вашу помощь!

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Oct 29, 2023
@javascript-translate-bot
Copy link
Contributor

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

@WOLFRIEND
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Oct 30, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team October 30, 2023 13:55
@WOLFRIEND
Copy link
Contributor Author

Добрый день,

Я внёс изменения в соответствии с вашими правками и ответил в комментариях.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs +1 One more review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants