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

feat: addWebpackExternals #84

Merged
merged 2 commits into from
Apr 21, 2019
Merged

Conversation

zhigang1992
Copy link
Contributor

No description provided.

readme.md Outdated
@@ -150,6 +150,10 @@ Adds the provided alias info into webpack's alias section. Pass an object litera

Adds the provided resolve info into webpack's resolve section. Pass an object literal with as many entries as you'd like, and the whole object will be merged in.

### addWebpackExternals(deps)

Add external dependencies, useful when trying to offload libs to CDN
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May you provide a short example to explain how to use this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally the example should be written directly in the document

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also firebase js sdk https://firebase.google.com/docs/web/setup#reserved-urls with externals we can import firebase in js file also using the js firebase provided

@zhigang1992
Copy link
Contributor Author

@FezVrasta docs added. 😄

@FezVrasta FezVrasta merged commit 3046d39 into arackaf:master Apr 21, 2019
@FezVrasta
Copy link
Collaborator

Great thanks!

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.

2 participants