Skip to content
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

Build reproducibility: pull fonts from npm #522

Open
avivace opened this issue Dec 17, 2023 · 3 comments
Open

Build reproducibility: pull fonts from npm #522

avivace opened this issue Dec 17, 2023 · 3 comments
Labels

Comments

@avivace
Copy link
Member

avivace commented Dec 17, 2023

We switched from CDNs to serving typefaces as static assets. When a new version of the font is released, this requires the manual step of pulling fonts builds from their upstream websites, clean up unrelated variants, remove them from their entrypoint CSS (e.g. inter.css) and commit the changed files.

This could be automated and made reproducible by switching to a package manager (e.g. npm) to pull the typefaces at build time.

E.g. inter on npm: https://www.npmjs.com/package/inter-ui

@avivace avivace added the mdbook label Dec 17, 2023
@ISSOtm
Copy link
Member

ISSOtm commented Dec 17, 2023

I'd prefer if this was not done unconditionally at build time, and instead there was a script to update the font file which is otherwise cached in the source.

This cuts back on the number of deps strictly required to run the site, as well as build times, at the cost of a comparatively modest amount of storage.

@avivace
Copy link
Member Author

avivace commented Dec 17, 2023

Agree!

@ISSOtm
Copy link
Member

ISSOtm commented Dec 17, 2023

Do you mind writing the steps down, at least? We can figure out how to script them separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants