Skip to content

vue-loader tells me I want vue-template-compiler, vue-template-compiler tells me I do not #1494

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

Closed
Julian opened this issue Feb 16, 2019 · 2 comments · Fixed by vuejs/vue#9508

Comments

@Julian
Copy link

Julian commented Feb 16, 2019

Version

15.6.0

Reproduction link

http://example.com

Steps to reproduce

This is essentially a doc bug --

As a new user, reading the docs from vue-loader tells me to install vue-template-compiler:

https://vue-loader.vuejs.org/guide/#manual-setup

vue-template-compiler on the other hand, tells me not to install it:

https://www.npmjs.com/package/vue-template-compiler

"You will only need it if you are writing build tools with very specific needs. In most cases you should be using vue-loader or vueify instead, both of which use this package internally."

So which is it (obviously I can see that likely vue-loader's docs are the correct ones, and that it doesn't declare the transitive dep for the reasons I see documented, but then vue-template-compiler shouldn't warn me not to install it)

What is expected?

Likely removing the warning from vue-template-compiler's page.

What is actually happening?

Not that.


[META] -- clearly the "Please do not just fill in a random link." is inappropriate for this sort of bug, so I've ignored it and done exactly that, but if you're going to create a custom way to file issues it might be nice to support these as well.

@ulivz
Copy link
Member

ulivz commented Feb 16, 2019

I just create a pull request to update the doc: vuejs/vue#9508

@Julian
Copy link
Author

Julian commented Feb 16, 2019

@ulivz that was quick! Thanks! Will keep an eye on it.

yyx990803 pushed a commit to vuejs/vue that referenced this issue Mar 1, 2019
kiku-jw pushed a commit to kiku-jw/vue that referenced this issue Jun 18, 2019
Lostlover pushed a commit to Lostlover/vue that referenced this issue Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants