-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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 vite peerDep warning by updating @joshwooding/vite-plugin-react-docgen-typescript
#20359
Fix vite peerDep warning by updating @joshwooding/vite-plugin-react-docgen-typescript
#20359
Conversation
@joshwooding/vite-plugin-react-docgen-typescript
@joshwooding Any idea why the UI changes might appear? Did anything else upgrade? |
Looks like it's actually broken: https://www.chromatic.com/test?appId=630511d655df72125520f051&id=63a2cd9f194ba874e9861c3e 😢 |
@ndelangen @joshwooding looks like the plugin is broken in this PR? |
Yes, looks that way, upsetting... I do not know what's wrong.. |
After a bit of debugging I found the problem, it was an CJS => ESM thing. I'll open a PR @joshwooding |
Cool, I saw it was that ts.createProgram isn't a function because the async import returned a default object |
@shilman self merging |
Issue: #20179
What I did
The peerDependency issue was resolved upstream, this upgrades the dependecy to a version that no longer has the issue.
see: https://github.com/joshwooding/vite-plugin-react-docgen-typescript/blob/HEAD/CHANGELOG.md