activewidgets / getlibs

OBSOLETE, DO NOT USE: This project is no longer maintained
MIT License
165 stars 12 forks source link

way to permanently disable cdnjs ? #8

Open billiegoose opened 6 years ago

billiegoose commented 6 years ago

Someone must have recently added module (react-contextmenu) to cdnjs recently, because it broke my app. It used to fetch react-contextmenu/dist/react-contextmenu.js from unpkg.com but once the cdnjs version was uploaded it tried to get it from cdnjs but that path /dist/react-contextmenu.js didn't work because cdnjs doesn't copy the package folder structure like unpkg does.

Or at least I think that's what happened. Anyhoo... I really have no love for cdnjs because I find unpkg much more predictable and if you've got some spare cycles I'd love a way to just disable the cdnjs option bc it's only caused me trouble. :)

billiegoose commented 6 years ago

I mean, obviously my app breaking is my fault - I'm not trying to make it production quality at the moment - I just mean this as feedback about what is working and what isn't for me with getlibs, and cdnjs as the default hasn't been working well for me.

xkam commented 6 years ago

The idea behind cdnjs was that the packages there were specifically made for a browser and therefore have better chances to run than average npm code which may or may not have a dist folder and 'browser' entry in package.json. Maybe this is no longer the case - can you give me more examples where cdnjs package did not do what you want?