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

feat: Заполнять токены режима compact значениями из regular, если они не заданы #1069

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

andrey-medvedev-vk
Copy link
Contributor

@andrey-medvedev-vk andrey-medvedev-vk commented Sep 13, 2024

Перед отправкой этого реквеста на ревью убедитесь, что:

  • в вашей ветке работает сборка (npm run build:local),
  • в вашей ветке проходят тесты (npm test),
  • в вашей ветке проходит линтер (npm run lint), некоторые ошибки можно автоматически поправить с помощью npm run lint:fix,
  • покрытие тестов не ниже минимального значения,
  • если вы вносите изменения в задокументированные части библиотеки,
    ваш pull request содержит обновления документации,
  • если вы вносите изменения в сами токены, вы
    согласовали их с дизайнерами.

Подробнее о внесении изменений в репозиторий токенов

To make sure compact mode is always filled with values,
even though it is not explicitly presented
Because we now have compact mode filled with values
Copy link

github-actions bot commented Sep 13, 2024

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
91.98% (+0% 🔼)
1205/1310
🟢 Branches
89.01% (+0.06% 🔼)
332/373
🟢 Functions
92.93% (+0.08% 🔼)
171/184
🟢 Lines
91.67% (+0.01% 🔼)
1123/1225
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟢
... / compileBreakpointsCssVarsDeclaration.ts
95.56% (-2.12% 🔻)
85% (-4.47% 🔻)
100%
95.35% (-2.21% 🔻)

Test suite run success

463 tests passing in 37 suites.

Report generated by 🧪jest coverage report action from cd41e9c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant