-
Notifications
You must be signed in to change notification settings - Fork 917
chore: add missing dependencies #1336
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
Conversation
Does |
@yyx990803 Done. |
Yes, it does allow accessing |
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.
I don't think memory-fs
and source-map
are necessary as they are added to devDependencies
. It was already working without these then there is no reason to change.
Thanks for adding webpack
as peer dependency.
@znck They were required to pass the tests with pnpm. In other words, they were "require"d from the tests transitively before, which is not good because you don't control the version you get. |
Vue is unusable without |
There are merge conflicts now |
* https://github.com/vuejs/vue-loader: (23 commits) chore: changelog 15.6.0 feat: make `__file` injection opt-in in production (vuejs#1475) fix: template comments replace windows \ to / confirm consistent hash (vuejs#1477) docs(zh): updated eslint plugin link (vuejs#1471) chore: todo [ci skip] feat: support for compiler 2.6 outputSourceRange feat: support webpack 5 hooks (vuejs#1469) docs: [RU] Translation update (vuejs#1470) chore: changelog 15.5.1 fix: avoid to generate empty css chunk files (vuejs#1464) chore: changelog 15.5.0 feat: add `prettify` option (vuejs#1461) chore: add missing dependencies (vuejs#1336) fix: css modules extends error (vuejs#1452) docs: [RU] Translation update (vuejs#1448) Update eslint-plugin-vue documentation link (vuejs#1444) docs: remove unnecessary `path` module usage (vuejs#1430) ...
Required for pnpm compatibility.
For full compatibility, vuejs/vue-style-loader#24 is also needed.