Repository navigation
perf(ci): prebuilt actionlint, skip input for rust-quality, run-speed rules - #136
Merged
Merged
Conversation
The actionlint Docker action rebuilt its image on every run (about 30 s). The release binary is checksum-pinned and takes about 2 s. rust-quality runs tests with cargo-nextest, which overlaps test binaries that cargo test runs one after another, plus a separate doctest pass. A new skip input lets callers skip the job for documentation-only changes while the required check name still reports.
Measured on pull request runs: opy-rs Run tests went from 52 s to 149 s, athena from 40 s to 65 s. Per-test processes cost more than overlapping the test binaries saves. The skip input stays.
Cancel superseded pull request runs only, filter documentation-only changes inside the workflow so required checks still report, avoid needs without data, count jobs, prefer prebuilt tools, and keep cargo test.
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
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.
Shared half of the CI speed-up.
actionlint.yml: checksum-pinned release binary instead of the Docker action that rebuilt its image on every run (about 30 s to about 5 s); sparse checkout; PR-only cancel.rust-quality.yml: newskipinput. A skipped job still reports<caller> / Qualityas skipped, which satisfies a required check, so callers can filter documentation-only changes without leaving required checks pending.docs/ci-platform.md: records the run-speed rules, including whycargo nextestwas rejected after measurement.Merge this before the consumer PRs; they reference this branch until it merges.