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

Микро правки оформления в статье про Deno #5572

Merged
merged 6 commits into from
Dec 16, 2024

Conversation

cergmin
Copy link
Contributor

@cergmin cergmin commented Dec 5, 2024

Статья интересная, но сходу начал резать глаза дефис вместо тире, не смог пройти мимо :)

Описание

  • Заменияет дефис на тире там, где это нужно
  • Выделяет некоторые фрагменты как код через бэктики

Чек-лист

  • [ x ] Текст оформлен согласно руководству по стилю
  • [ x ] Ссылки на внутренние материалы начинаются со слеша и заканчиваются слэшем либо якорем на заголовок (/css/color/, /tools/json/, /tools/gulp/#kak-ponyat)
  • [ x ] Ссылки на картинки, видео и демки относительные (images/example.png, demos/example/, ../demos/example/)

Статья интересная, но сходу начал резать глаза дефис вместо тире, не смог пройти мимо :)
@github-actions github-actions bot added веб-платформа Контент по Веб-платформе статья Расширенный материал labels Dec 5, 2024
@vitya-ne vitya-ne added the улучшение Доработка существующего label Dec 5, 2024
Copy link
Member

@Inventoris Inventoris left a comment

Choose a reason for hiding this comment

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

Привет! Крайне добротно, спасибо! Предлагаю несколько правочек.

@cergmin
Copy link
Contributor Author

cergmin commented Dec 5, 2024

@Inventoris, закоммитил твои правки

Copy link
Member

@Inventoris Inventoris left a comment

Choose a reason for hiding this comment

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

Здорово! ╰(°▽°)╯

@solarrust
Copy link
Member

solarrust commented Dec 6, 2024

Чтобы два раза не вставать =)

  1. Добавила пару ссылок.
  2. Заменила список на ленивый.
  3. Сделала имена файлов курсивом.
  4. Заменила кавычки в JS-коде на одинарные, убирает точки с запятой.
  5. Вынесла комменты из кода там, где это возможно.

Пожалуйста, дай знать, если какая-то из моих правок, на твой взгляд, нарушила суть текста. Всё обсуждаемо =)

@cergmin
Copy link
Contributor Author

cergmin commented Dec 7, 2024

Раз уж этот фрагмент начали править :)

Перенёс команду вызова deno выше кода файла, который мы запускаем, потому что:

  1. Это маленькая строчка совсем теряется после большого полотна
  2. Она мне кажется здесь более важной, так как именно про неё мы говорим в абзаце выше «мы должны запустить скрипт из терминала с флагами»

@solarrust, что думаешь?

  1. Вынесла комменты из кода там, где это возможно.

Субъективно мне больше нравилось, когда название файла было указано в коде, так как в тексте оно теряется на мой взгляд, но это не принципиально. Идеально было бы, если бы платформа Доки позволяла указать название файла рядом с языком, но пока такого нет, кажется.

Copy link

github-actions bot commented Dec 7, 2024

Превью контента из a0e73f4 опубликовано.

@solarrust solarrust merged commit d3bef28 into doka-guide:main Dec 16, 2024
6 checks passed
baileys-li pushed a commit to baileys-li/doka-content that referenced this pull request Dec 19, 2024
* Микро правки оформления в статье про Deno

Статья интересная, но сходу начал резать глаза дефис вместо тире, не смог пройти мимо :)

* Оборачивает require() в бэктики

Co-authored-by: Alexey Nikitchenko <[email protected]>

* Пишет Deno с большой буквы

Co-authored-by: Alexey Nikitchenko <[email protected]>

* Оборачивает async/await в бэктики

Co-authored-by: Alexey Nikitchenko <[email protected]>

* Немного форматирует и редактирует

* Выносит наверх более важную часть (строчку с запуском файла)

---------

Co-authored-by: Alexey Nikitchenko <[email protected]>
Co-authored-by: Alena Batitskaia <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
веб-платформа Контент по Веб-платформе статья Расширенный материал улучшение Доработка существующего
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants