-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Provide pre-minified code #61
Comments
sounds good. shall we do a minified |
Getting the lib from google code or github raw isn't optimal (bad headers, bad caching, no compression). Wouldn't it be better to it up at http://www.cdnjs.com? (or update, seems an old version lives there) |
@bjornharrtell in html5shiv's case you basically DEFINITELY want the latest version always. we dont want people to manually update. currently cdnjs and google's cdn can't support this model very well, so IMO it makes more sense to host it locally |
Let's be honest, most web devs are going to copy and paste. I'm in the process of updating the googlecode repos to the latest code (I kinda thought I was supposed to hold off releasing "pre" - but it sounds like it's stable).
Anyway, I've remove the hotlinking, forcing users to now download - but we're lazy users and they'll want the easiest solution otherwise a lot will not bother. I'm offering up a pre-minified version of the code here, but it strikes me if I can link directly to pre-minified code on this repo - then if the user hits download from the googlecode site - it'll land here (a good thing I think we can agree).
The text was updated successfully, but these errors were encountered: