Closed carlosperate closed 2 years ago
@JohnVidler Looks like the CI is no longer running on PRs.
I've temporarily disabled PR builds to prevent excessive runner instances. Just got to fix the docker action (the artifacts aren't pulled correctly even though the build succeeds, which then fails the test...) then I'll reenable the PR trigger and disable the manual one.
Later down the line I'd like to have a few distinct build configurations to compile for different (ideally all) example codebases, so we have better test coverage for all CODAL components, but that's for another PR, I think 😁
I've re-enabled pull request actions and both the Docker and native builds are working fine now.
As the Docker build uses a static set of tooling (as it should do) this patch doesn't currently touch it, but it would be nice to have a future one which updated the Docker in-line with the latest tooling we use for the native builds.
Nonetheless, happy to merge this in now that the base builds are working.
Now that CODAL builds with the latest GCC versions we can change the CI config to also build with the latest available release (together with an older one). This should help to catch earlier any issues with new GCC releases.