-
Notifications
You must be signed in to change notification settings - Fork 1
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
CDN #1
Comments
Very cool! 👍 Is it purely out of pursuit of neatness that we move all wanted files to this repo instead of just using their own jsDelivr links? e.g.: https://cdn.jsdelivr.net/gh/LingDong-/wenyan-book/assets/font.woff2 |
Yeah, I think the font subset is ok to be in the repo and use is directly by jsDelivr. (Please also aware that The main reason this repo gets introduced is that files like those are CI/CD on every commit automatically. And for most of the time, you don't want to commit these files into source control and manually rebuild it repeatedly. And yeah, maybe another way is to commit files into a different branch of the source repo instead this centralized repo. |
FYI, I have moved the dist files to the |
👍 Sounds good. Thanks! |
I create this repo for hosting files for CDNs. I am planning to use different branches for different projects.
jsDelivr has an awesome feature that direct host files from Github repos, and surprisingly it's kinda fast in China.
So I moved files that originally hosted by Netlify to this repo by a Github Action:
https://wy-lang.org/dist/core.js
(somehow it's very slow in China)now becomes:
https://cdn.jsdelivr.net/gh/wenyan-lang/cdn@wenyan-core/core.min.js
(and with the.min.js
suffix, jsDelivr will automatically minimize for you)I also changed the CDN for IDE in wenyan-lang/ide@3f03542
So far it's just a quick experiment and I think it works well. In the future, you may consider moving fonts for wenyan-book by these steps to get better loading speed for users in China. :)
The text was updated successfully, but these errors were encountered: