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

Hoisting removed ? #48

Open
recrsn opened this issue Apr 10, 2016 · 1 comment
Open

Hoisting removed ? #48

recrsn opened this issue Apr 10, 2016 · 1 comment

Comments

@recrsn
Copy link

recrsn commented Apr 10, 2016

Came back to JavaScript development after a while. Noticed that change 90a7e6d defeats the purpose of CDNify in some yeoman generated projects. Yeoman's gruntfile uses google-cdn to before usemin to change references to commonly used libraries (such as Angular in my case). After replacements, the entire scripts block is replaced, along with the references changed by google-cdn. As a result all references to libraries is lost.

Can the support for hoisting be retrofitted ?

@david-mohr
Copy link
Contributor

I put together a gulp package to help with this: https://github.com/david-mohr/gulp-cdn-hoist (based on the grunt package https://github.com/mjetek/grunt-cdn-nobuild)

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

No branches or pull requests

2 participants