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

[Vue] Update rollup config #17863

Merged
merged 2 commits into from Mar 25, 2019
Merged

Conversation

michaeltintiuc
Copy link
Contributor

Fixes rollup warnings:

(!) Mixing named and default exports
Consumers of your bundle will have to use bundle['default'] to access the default export, which may not be what you want. Use `output.exports: 'named'` to disable this warning
(!) Unresolved dependencies
https://github.com/rollup/rollup/wiki/Troubleshooting#treating-module-as-external-dependency
@ionic/core (imported by src/controllers/vue-delegate.ts)

@ionitron-bot ionitron-bot bot added the package: vue @ionic/vue package label Mar 22, 2019
@mhartington mhartington merged commit c148d31 into ionic-team:master Mar 25, 2019
kiku-jw pushed a commit to kiku-jw/ionic that referenced this pull request May 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: vue @ionic/vue package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants