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.

Sign up for GitHub

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

Jump to bottom

JS: Исправляет ошибки и форматирует (часть 10) #5377

Merged
merged 6 commits into from
Jul 8, 2024
Merged

JS: Исправляет ошибки и форматирует (часть 10) #5377

merged 6 commits into from
Jul 8, 2024

Conversation

TatianaFokina
Copy link
Member

Описание

Обнаружила в object неиспользуемые демки. Они вообще не встроены что в основной текст, что в текст совета.

Чек-лист

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

@TatianaFokina TatianaFokina added улучшение Доработка существующего ошибка Что-то критичное и мешает labels Jun 10, 2024
@TatianaFokina TatianaFokina self-assigned this Jun 10, 2024
@github-actions github-actions bot added js Контент по JavaScript статья Расширенный материал дока Справочный материал placeholder Заготовка материала labels Jun 10, 2024
skorobaeus
skorobaeus approved these changes Jun 11, 2024
View reviewed changes
Copy link
Member

@skorobaeus skorobaeus left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Демки в object очень странные, там везде предлагается взглянуть в консоль (мы такое уже не носим), и, при этом, подводок к ним в тексте нет. Пусть лежат, видимо, есть-пить не просят; вдруг когда-нибудь кто-нибудь поймёт, что с ними делать.

js/null-primitive/index.md Outdated Show resolved Hide resolved
js/number-is-nan/index.md Outdated Show resolved Hide resolved
js/number-is-nan/practice/nlopin.md Outdated Show resolved Hide resolved
js/number/index.md Outdated Show resolved Hide resolved
js/number/index.md Outdated Show resolved Hide resolved
js/number/index.md Outdated Show resolved Hide resolved
js/number/index.md Outdated Show resolved Hide resolved
js/object/practice/doka-dog.md Outdated Show resolved Hide resolved
@solarrust
Copy link
Member

Я за то, чтобы не тянуть всё, что не используется. Каждый файлик тянется при загрузке репозитория. Давай удалим?

HellSquirrel
HellSquirrel reviewed Jul 1, 2024
View reviewed changes
js/number-isfinite/practice/windrushfarer.md Outdated Show resolved Hide resolved
js/object/index.md Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 8, 2024

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

@TatianaFokina TatianaFokina merged commit 470b142 into doka-guide:main Jul 8, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skorobaeus skorobaeus skorobaeus approved these changes

@vitya-ne vitya-ne vitya-ne left review comments

@HellSquirrel HellSquirrel HellSquirrel left review comments

@solarrust solarrust solarrust approved these changes

Assignees

@TatianaFokina TatianaFokina

Labels
js Контент по JavaScript placeholder Заготовка материала дока Справочный материал ошибка Что-то критичное и мешает статья Расширенный материал улучшение Доработка существующего
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@TatianaFokina @solarrust @HellSquirrel @vitya-ne @skorobaeus