search

VKCOM / icons

Набор SVG иконок, представленный в виде React компонентов.
https://vkcom.github.io/icons/
MIT License
130 stars 69 forks source link

BREAKING CHANGE: new react runtime #902

Open SevereCloud opened 3 months ago

SevereCloud commented 3 months ago

Устанавливаем новый рантайм, меняем поддержку реакта на ^18

github-actions[bot] commented 3 months ago

👀 Docs deployed

See the docs for this PR at https://vkcom.github.io/icons/pull/902/

SevereCloud commented 3 months ago

size как-то не правильно сработал

mendrew commented 3 months ago

size как-то не правильно сработал

Да, там теперь не хватает пересборки icons-sprite, так как многие зависимости поменялись, а результат сборки используется в icons-build.

Нужен yarn workspace @vkontakte/icons-sprite build перед запуском icons-build. Надо это, похоже в одну команду icons-build запихнуть, причём в отдельном PR, а то билд сыпется на мастере, который про изменения в этой ветке не узнает.

mendrew commented 3 months ago

@SevereCloud поменял вызов icons-build, чтобы size-limit не падал на этом PR https://github.com/VKCOM/icons/pull/909

Протестировал тут как раз с твоими изменениями: https://github.com/VKCOM/icons/pull/910

github-actions[bot] commented 3 months ago

size-limit report 📦

Path Size
JS 3.93 MB (-0.01% 🔽)
JS (gzip) 968.44 KB (-0.01% 🔽)
JS (brotli) 695.65 KB (+0.01% 🔺)
JS ES6 with Icon16Add only import (tree shaking) 5.36 KB (-5.45% 🔽)
SVG 3.78 MB (0%)