-
-
Notifications
You must be signed in to change notification settings - Fork 198
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
fix(common): Move typing dependencies to peer #477
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Ignored Deployments
|
Something that could be interesting, is doing the same thing with form-render, but I am unsure if form-render is a real dependency |
The tests are passing with form-render as peer as well. Should I do additional checks? |
Hey @sebimarkgraf ! Sorry for the delayed response. |
Hi @hand-dot, any update on this MR, thank you. |
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.
LGTM!
Hey @sebimarkgraf Sorry for the delayed review. Thank you for your contribution! |
Released as V4.0.2!! |
This could be a fix for #474 and the other related issues.
This ignores the type import of antd and casts it to any if antd is not installed.
Additionally, the antd dependency is downgraded to peer in common.
If ui is installed the direct dependency would lead to the correct typing being filled in.