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

Добавляет в документацию инструкцию о подключении Baseline #5689

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open

Добавляет в документацию инструкцию о подключении Baseline #5689

wants to merge 27 commits into from
+107 −7

Conversation

vitya-ne
Copy link
Contributor

@vitya-ne vitya-ne commented Feb 8, 2025

Описание

Как смог описал процесс добавления baseline в статью. Готов к корректировкам. )

Closes #5019

Чек-лист

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

@vitya-ne vitya-ne added улучшение Доработка существующего кухня Документация, репозиторий, задачи редакции labels Feb 8, 2025
@vitya-ne vitya-ne self-assigned this Feb 8, 2025
@vitya-ne vitya-ne changed the title Добавляет в документацию инструкцию о пдключении Baseline Добавляет в документацию инструкцию о подключении Baseline Feb 8, 2025
nasty23-star
nasty23-star requested changes Feb 9, 2025
View reviewed changes
Copy link
Contributor

@nasty23-star nasty23-star left a comment

Choose a reason for hiding this comment

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

Большое спасибо за материал, всё здорово расписано! \О_О/
совсем немножко доработаем и будет огонь-пожар!

furtivite
furtivite requested changes Feb 11, 2025
View reviewed changes
Copy link
Member

@furtivite furtivite left a comment

Choose a reason for hiding this comment

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

Получилось очень круто. Я внёс несколько предложений по тексту. Возможно, это натолкёт на мысли про стиль повествования.

@vitya-ne
Copy link
Contributor Author

@furtivite, @nasty23-star
Спасибо за коменты, поправил вроде бы всё.

@furtivite
Copy link
Member

@furtivite, @nasty23-star Спасибо за коменты, поправил вроде бы всё.

Круто получилось, я попрошу кого-нибудь из редакции посмотреть дополнительно. Спасибо)

@furtivite furtivite self-requested a review February 11, 2025 20:18
@igsekor igsekor self-requested a review February 12, 2025 09:18
igsekor
igsekor requested changes Feb 12, 2025
View reviewed changes
Copy link
Member

@igsekor igsekor left a comment

Choose a reason for hiding this comment

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

В целом всё круто!❤️Есть два нюанса, я их описал.

Inventoris
Inventoris reviewed Feb 14, 2025
View reviewed changes
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.

Витя привет!

Круто, этого руководства не хватало. Спасибо тебе! 💫

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

vitya-ne and others added 4 commits February 14, 2025 21:28
@vitya-ne @igsekor
Update docs/baseline.md
88bfce1 
Co-authored-by: Igor Korovchenko <[email protected]>
@vitya-ne @Inventoris
Update docs/baseline.md
6f0ffce 
Co-authored-by: Alexey Nikitchenko <[email protected]>
@vitya-ne @Inventoris
Update docs/baseline.md
603d5c7 
Co-authored-by: Alexey Nikitchenko <[email protected]>
@vitya-ne @Inventoris
Update docs/baseline.md
df34f89 
Co-authored-by: Alexey Nikitchenko <[email protected]>
solarrust
solarrust reviewed Feb 17, 2025
View reviewed changes
Copy link
Member

@solarrust solarrust left a comment

Choose a reason for hiding this comment

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

Как же долго я ждала этот документ ♥️

Оставила очень маленькие предложения по улучшению.

@vitya-ne
Copy link
Contributor Author

@Inventoris, @igsekor, @solarrust,
Поправил как мог, посмотрите, пожалуйста

Inventoris
Inventoris approved these changes Feb 18, 2025
View reviewed changes
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.

Класс! Давай добавим ещё пару правочек, но от меня уже апрув!

(✧×✧)

vitya-ne and others added 3 commits February 18, 2025 20:52
@vitya-ne @Inventoris
Update docs/examples/doka.md
3a955a3 
Co-authored-by: Alexey Nikitchenko <[email protected]>
@vitya-ne @Inventoris
Update docs/baseline.md
6cad754 
Co-authored-by: Alexey Nikitchenko <[email protected]>
@vitya-ne @Inventoris
Update docs/baseline.md
36acac6 
Co-authored-by: Alexey Nikitchenko <[email protected]>
@github-actions GitHub Actions
Copy link

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

@vitya-ne
Copy link
Contributor Author

@igsekor, @solarrust, пинг

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@solarrust solarrust solarrust left review comments

@igsekor igsekor igsekor requested changes

@furtivite furtivite furtivite approved these changes

@nasty23-star nasty23-star nasty23-star approved these changes

@Inventoris Inventoris Inventoris approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

@vitya-ne vitya-ne

Labels
кухня Документация, репозиторий, задачи редакции улучшение Доработка существующего
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Добавить baseline в документацию
6 participants
@vitya-ne @furtivite @solarrust @igsekor @nasty23-star @Inventoris