fix: don't require optional peerDependencies #2197
Conversation
✅ Deploy Preview for vue-test-utils-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
`@vue/server-renderer` and `@vue/compiler-dom` were both required in the sense that they will throw regardless of whether or not your are using functionality from those libraries if they weren't installed. I don't believe this was intentional, so I have made them optional and they will now only throw if you try to use them and the package isn't available, and will print a more helpful error message when doing so.
thebanjomatic
force-pushed
the
fix/peer-deps
branch
from
4212a60
to
e0585e4
Compare
thebanjomatic
changed the title
|
How can I reproduce and test the issue this supposedly fixes? I am not comfortable merging this as it - I don't know what it fixes or how to test it. |
|
I'll see if I can create a small reproducer. |
|
I created a test repo that demonstrates 3 separate failures across yarn, pnpm and npm that are related to the issue in this PR. I was really hoping to just see the same problem manifest itself in the same way 3 times, but funny enough each package manager seems broken in slightly different ways. |
|
Today (after updating test-utils) I ran into the issue fixed by this PR. For now my workaround is to add hoist-pattern[]=@vue/server-rendererto my |
@vue/server-rendererand@vue/compiler-domwere both required in thesense that they will throw regardless of whether or not your are using
functionality from those libraries if they weren't installed. I don't
believe this was intentional, so I have made them optional and they will
now only throw if you try to use them and the package isn't available,
and will print a more helpful error message when doing so.