Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

jsDelivr complains about package size exceeded the configured limit of 50 MB #1510

Closed
kevinsimper opened this issue Aug 15, 2018 · 4 comments
Labels
exp/novice Someone with a little familiarity can pick up help wanted Seeking public contribution on this issue P2 Medium: Good to have, but can wait until someone steps up

Comments

@kevinsimper
Copy link

Package size exceeded the configured limit of 50 MB

I think it is important as links in the readme are to jsDelivr

https://www.jsdelivr.com/package/npm/ipfs

image

@alanshaw alanshaw added exp/novice Someone with a little familiarity can pick up help wanted Seeking public contribution on this issue status/ready Ready to be worked P2 Medium: Good to have, but can wait until someone steps up labels Aug 16, 2018
@mikeal
Copy link
Contributor

mikeal commented Aug 16, 2018 via email

@MartinKolarik
Copy link

Hi, Martin from jsDelivr here. Technically we are able to serve packages even if they exceed this limit and I could add an exception for ipfs.

The reason for having a limit by default is that npm only provides tarballs with all files so we always have to fetch everything, even if nothing else than dist/ is needed. The bigger the package, the higher the chance this takes too long and some requests occasionally fail.

We also find that very few packages actually need to exceed this limit. In your case, 63 MB out of 80 MB are just the examples. In my opinion, those do not belong to the npm package at all and should be added to .npmignore.

@alanshaw
Copy link
Member

fixed in #1513 and released in 0.31.7

@ghost ghost removed the status/ready Ready to be worked label Aug 20, 2018
@kevinsimper
Copy link
Author

@alanshaw Awesome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
exp/novice Someone with a little familiarity can pick up help wanted Seeking public contribution on this issue P2 Medium: Good to have, but can wait until someone steps up
Projects
None yet
Development

No branches or pull requests

4 participants