-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add GS main branch build CI testing #574
Conversation
8f0c4d7
to
98c120f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic, please merge once it passes :)
.github/workflows/gsbuild.yml
Outdated
|
||
build-italic: | ||
runs-on: ubuntu-latest | ||
name: Build Italic variable font |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If compiling the compiler is by far the slowest step it's possible that building Romand and Italic in one job would make sense?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the current approach the fontc compile takes about 50s and each variable font compile takes 1.5 mins. OK to keep this sequential approach or should we run the builds in parallel again?
98c120f
to
33419c2
Compare
8f59562
to
7db4cb1
Compare
Adds a GH Action workflow to build the GS project on pull request commits with a debug build of the fontc compiler.