Run CI test groups on ubuntu only (drop the os matrix)#71
Merged
ChrisRackauckas merged 2 commits intoJun 10, 2026
Conversation
Aqua/JET QA checks are platform-independent, so running them on windows-latest and macos-latest was wasted CI. Remove the os axis from the [QA] group so it runs only on the default ubuntu-latest. Core keeps its full multi-OS matrix. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com> Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…pecific Pure-Julia package with no platform-dependent code paths, artifacts, or binary deps; macos/windows Core runs add CI cost without coverage. The windows grouped jobs currently do not even run their declared group (SciML/.github#81), so their green status was not real coverage anyway. Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
ChrisRackauckas-Claude
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The QA test group (Aqua/JET) was running on the full multi-OS matrix (ubuntu, macos, windows). Those checks are platform-independent, so the windows/macos QA jobs were wasted CI with no added coverage.
Extended per review feedback: the
osaxis is now dropped from all groups (Core included). This is a pure-Julia package — no platform-dependent code paths, no artifacts, no binary deps beyond stdlib — so per-OS CPU test runs add cost without coverage. With theoskey absent, the grouped-tests workflow defaults each job toubuntu-latest.Two additional data points in favor:
tests.ymllacksshell: bash, soGROUPis never set on windows and every windows job silently runs the default group (fix: tests.yml: set test group env with shell: bash so it takes effect on Windows .github#81). Their green checkmarks were not real coverage.Restoring an OS for any group later is one line in
test/test_groups.toml.Ignore until reviewed by @ChrisRackauckas.
🤖 Generated with Claude Code