Milligram: CDN link broken

Created on 6 Jun 2020  ·  6Comments  ·  Source: milligram/milligram

Version info

Milligram: Latest (From CDN)

Other (e.g. normalize.css, node.js, npm, bower, browser, operating system) (if applicable):

Test case

Using the link from the "Getting started" page: cdn.rawgit.com/milligram/milligram/master/dist/milligram.min.css, it's a 404.

Steps to reproduce

  1. Install Milligram via CDN links in the getting started guide
  2. Observe that Milligram CSS is a 404

Expected behavior

A 200 followed by the CSS for milligram

Actual behavior

A 404

It looks like /dist was removed from this repo some time ago but rawgit.com only finally purged it from cache. I'm not sure when since this is on a site I don't look at much, maybe within the last month.

For CDNs, rawgit.com is also going away. The rawgit.com page lists some alternatives.

All 6 comments

It looks like the dist folder was removed in this commit on June 1.

@cjpatoilo is there any chance you might consider restoring the dist folder with the (minified and unminified) CSS source at some point in the future? (For what it's worth, this also broke our ability to update milligram automatically at dropin-minimal-css.)

@swalberg @dohliam

Please, can you review this pull request https://github.com/milligram/milligram/pull/251

@cjpatoilo Thanks so much for the quick fix! It's great to have the compiled CSS available again. :+1:

Thank you! Looking forward to seeing this merged!

@swalberg @dohliam it's my pleasure to rollback the Rawgit support. So, I just didn't know that there was this problem. I love Rawgit and @rgrove made the huge work.

@swalberg @dohliam all done! Now I will close this issue and if you have any questions, please, feel free to reopen this issue or open a new one.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

roryprimrose picture roryprimrose  ·  7Comments

faststare picture faststare  ·  4Comments

piever picture piever  ·  3Comments

kresogalic8 picture kresogalic8  ·  5Comments

dnohr picture dnohr  ·  7Comments