Skip to content

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

Draft
andrey-medvedev-vk wants to merge 5 commits intomasterfrom
mendrew/fallback-compact-mode-with-regular-values
Draft

feat: Заполнять токены режима compact значениями из regular, если они не заданы#1069
andrey-medvedev-vk wants to merge 5 commits intomasterfrom
mendrew/fallback-compact-mode-with-regular-values

Conversation

@andrey-medvedev-vk
Copy link
Copy Markdown
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
@github-actions
Copy link
Copy Markdown

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