-
Notifications
You must be signed in to change notification settings - Fork 182
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
[Docs] propsParser в production-сборке и под флагом #2151
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit b981e78:
|
size-limit report 📦
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Я бы дефолтным сделал typescript парсер. Хочется, чтобы условие было инвертировано. Типа: если дев, то собирай упрощенно, иначе собирай прод. Так меньше шансов, что однажды на прод выедет сборка без пропов.
- Если мы ваще не выводим пропы в упрощенном режиме, то можем ли мы не запускать никакой парсер? Или стайлгайдист не даст?
& VKUI_STYLEGUIDE_PROPSPARSER env var
bf4944a
to
3c1adb7
Compare
👀 Styleguide deployedSee the styleguide for this PR at https://vkcom.github.io/VKUI/pull/2151/ |
disable propsParser for dev
in dev mode
3c1adb7
to
b981e78
Compare
Из-за возвращения на
react-docgen-typescript
с dev-сборкой документации возникли две проблемы:При этом пропсы компонентов меняются не так часто и кажется, что необходимости ждать 2 минуты каждый раз нет.
Поэтому пока есть такое временное решение:
react-docgen-typescript
остался для prod-сборки,yarn styleguide:props
, чтобы в dev-сборке можно было использоватьreact-docgen-typescript
и посмотреть, какие пропсы и как выведутся в "Свойствах и методах", когда и если это нужно,Попозже и уже отдельно хочу попробовать дооптимизировать
react-docgen-typescript
так, чтобы танцы с бубном вокруг парсера были больше не нужны.