Task DS-549: [React Native] Поддержка UniStyles: Базовая конфигурация#37
Task DS-549: [React Native] Поддержка UniStyles: Базовая конфигурация#37ragozin-nikita wants to merge 6 commits into
Conversation
Добавил кеширование и игнорирование вспомогательных папок
Поднял количество maxWorker до максимума
ragozin-nikita
changed the title
| }, | ||
| "dependencies": { | ||
| "@tabler/icons-react-native": "^3.36.0" | ||
| "@tabler/icons-react-native": "3.36.1", |
There was a problem hiding this comment.
Здесь специально была не зафиксирована версия, чтобы меньше конфликтов было в случае, когда в проекте уже используется этот пакет. Считаю, что можно оставить ^3.36.1
| "dependencies": { | ||
| "@tabler/icons-react-native": "^3.36.0" | ||
| "@tabler/icons-react-native": "3.36.1", | ||
| "react-native-unistyles": "3.2.3" |
There was a problem hiding this comment.
Я не уверен, что нужно unistyles добавлять в dependencies, скорее в peerDependencies.
В проекте на момент подключения ui kit уже может использоваться unistyles, в таком случае непонятно как это будет работать при наличии unistyles в dependencies
| import type { ThemeType } from '../theme' | ||
|
|
||
| /** | ||
| * @deprecated Используйте `StyleSheet.create(...)` из SDK. |
There was a problem hiding this comment.
SDK - не очень понятно, что имеется в виду
There was a problem hiding this comment.
Удален весь deprecated код
| * | ||
| * @deprecated Используйте `useUnistyles().theme.fonts` или `StyleSheet.create(...)` | ||
| * из SDK. | ||
| * Будет удалён в следующей minor версии. |
There was a problem hiding this comment.
А мы решили вроде мажорную версию поднимать. Если так и будет, может и удалить сразу? А то как-то нелогично получается, в мажорной версии оставляем поддержку старого апи, а в минорной - убираем
There was a problem hiding this comment.
И еще момент вспомнил. Мы вроде хотели мажорную версию одинаковую поддерживать между всеми фреймворками и версия 1 должна по идее быть для либы с новыми токенами
Уточню еще у Данила что там у нас с версиями
UPD.: Уточнил - не проблема. Разные версии на вебе и мобилке - это ок
There was a problem hiding this comment.
Согласен, но в #38 этот хук не deprecated
|
Правки по текущему ревью перенесены в #38 |
2 participants
No description provided.