Skip to content

Conversation

@ilanbiala
Copy link

Users installing Bower don't need the src, and just want the jquery.js or jquery.min.js file.

Users installing Bower don't need the src, and just want the jquery.js or jquery.min.js file.
@mgol
Copy link
Member

mgol commented Mar 1, 2014

The src/ directory exists for people wanting to use jQuery modules as AMD modules; they should be included.

@mgol mgol closed this Mar 1, 2014
@ilanbiala
Copy link
Author

What about people who just want jquery.min.js? This adds a lot of clutter. I'm all for supporting everyone and it makes sense to include it if people use it, but maybe have a separate bower/node package then?

@mgol
Copy link
Member

mgol commented Mar 1, 2014

I don't understand, what do you mean by "a separate bower/node package"; what does it have to do with node?

Generally speaking, it would require us to create a separate repository just to publish only the jquery.js and jquery.min.js files; currently we publish from the main jquery repository. How should we name the package? We currently publish using the jquery-release script; this would have to be added to the script, every jQuery project would need to have a separate repository created, we would have to think about how to name the repositories... etc.

And what about the unminified file itself, isn't it clutter if you just want to get the minified file for production? After all, the unminified file takes about 244 KB. The whole src directory takes about 916 KB. If 916 KB is a lot for someone, it's highly probable they'll consider 244 KB to be a lot, too.

Anyway, jQuery is open source. If you feel there is a need for such separate smaller package, you can create & maintain one by yourself.

This was referenced Oct 29, 2014
@mgol mgol mentioned this pull request Mar 25, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jan 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants