-
Notifications
You must be signed in to change notification settings - Fork 20
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
CI Pre-build and download #115
Comments
Looks like we already have some CI in place. I am going to do some local testing with act to see if I can improve it, and get it building for more targets, then work on the entry-point to prefer a pre-built from CI over cmake-built. |
I started work on this with a fancier build-matrix. I also merged test & build to optimize cacheing, and set it to run on every push/PR, so we have a nice test indicator & build to play with on every platform. remaining issues
|
That's awesome! Nicely done. |
I added a stub setup in tools/postinstall.js. It will load very fast if you have |
We could speed up install with CI builds for lots of targets, and download on install, or just include many with the npm module.
I think some good targets would be:
and then, maybe as secondary targets:
On any targets not covered, it should work the same as it does now, so it's really just a way to speed it up (and not require cmake/c tools/build-time) on most the common platforms.
The text was updated successfully, but these errors were encountered: