Skip to content
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

Pass attributes to parseComponent (addresses #4914) #4925

Merged
merged 4 commits into from
Feb 14, 2017
Merged

Conversation

zephraph
Copy link
Contributor

@zephraph zephraph commented Feb 14, 2017

See #4914

This PR passes attributes down to parseComponent so that build tools layered on top of vue-template-compiler can have access to that information without having to parse the src to retrieve it.

@zephraph
Copy link
Contributor Author

Tests are incoming.

@yyx990803
Copy link
Member

The build.js is auto-genarated before publishing the package. Can you apply the change in src/sfc/parse.js instead?

@zephraph
Copy link
Contributor Author

Ahhh! I can't believe I missed that. >_> Sorry about that, yeah, I absolutely will.

@zephraph
Copy link
Contributor Author

@yyx990803 do you want me to undo the changes I made to build.js?

@yyx990803
Copy link
Member

yes please

@zephraph
Copy link
Contributor Author

zephraph commented Feb 14, 2017

done. Also, there's only one test... I can certainly add more if desired.

@zephraph
Copy link
Contributor Author

zephraph commented Feb 14, 2017

I added an extra check just for good measure. It's ready for review now.

@yyx990803 yyx990803 merged commit 97d36bc into vuejs:dev Feb 14, 2017
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 this pull request may close these issues.

3 participants