ci: select native validation by change impact - #80
Merged
Merged
Conversation
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
1 participant
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.
Release-policy tests and Markdown changes currently start the full native compiler matrix. Select compilation using a small explicit allowlist for documentation and release tooling; native tests, shared build inputs, packaging, workflows, submodules, and unknown paths retain all native lanes. Renames and deletions consider both paths. Both validation workflows report on every PR and main push, and the new wrapper result rejects failed, cancelled, missing, or unexpectedly skipped jobs.
Remove the duplicate Windows ARM64 v0.4.1 entry: the pinned revision is already the same b29c606e commit and runs identical commands. Keep the pinned check names and historical 73ab coverage because issue #76 remains open. Add cancellation only for superseded PR runs. Runtime pins, release/publication requirements, and repository settings are unchanged.
Validation: 124 Python tests, 18 artifact tests, actionlint, the release provenance verifier, and diff checks passed locally. Independent review found no blocking issues, including 8,192 aggregate-state cases and real Git rename/deletion tests. The workflow change selects the full hosted matrix; its results must pass before readiness.
Tracks leehack/llamadart#532. Savings depend on the changed paths: one duplicate Windows ARM64 build is removed from native runs, and known tooling/documentation changes avoid native compilation. No measured cost-reduction percentage is claimed.