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.

Sign up for GitHub

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

Jump to bottom

Don't depend on whole of bower #62

Open
sindresorhus opened this issue May 15, 2015 · 1 comment · May be fixed by #63
Open

Don't depend on whole of bower #62

sindresorhus opened this issue May 15, 2015 · 1 comment · May be fixed by #63
Assignees
@passy
Labels
enhancement

Comments

@sindresorhus
Copy link

grunt-google-cdn/tasks/cdnify.js

Line 6 in 5ddf769

var bowerConfig = require('bower').config;

The dependency is absolutely massive. Currently this grunt task with dependencies takes up 93 MB!

You can probably instead use https://github.com/bower/config

Related to passy/google-cdn#27
This was referenced May 15, 2015
Open
Closed
@passy
Copy link
Collaborator

passy commented May 16, 2015

93 MB. Wow. Thanks, Sindre. That's something we should be looking into.

@passy passy self-assigned this May 16, 2015
passy added a commit that referenced this issue May 21, 2015
@passy
Replace bower with bower-config
750d859 
Requires manual testing.
Fixes #62
@passy passy linked a pull request May 21, 2015 that will close this issue
Open
passy added a commit that referenced this issue Jun 12, 2015
@passy
fix(task): replace bower with bower-config
a1e92a9 
This should dramatically reduce the package size.

Requires manual testing.
Fixes #62
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@passy passy

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace bower with bower-config btford/grunt-google-cdn
2 participants
@passy @sindresorhus