Repository navigation
CI: consolidate script self-tests into one real-triggered workflow - #861
Closed
Brandon Miller (brandom-msft) wants to merge 1 commit into
Closed
Brandon Miller (brandom-msft) wants to merge 1 commit into
Brandon Miller (brandom-msft) wants to merge 1 commit into
Conversation
Replace the two inert *-selftest.yml dev harnesses (which only fired on PRs based on a throwaway dev branch, so never on real traffic) with one lean .github/workflows/scripts-selftest.yml that triggers on pull_request + paths: ['.github/scripts/**'] (plus workflow_dispatch). Jobs (contents:read only, non-required, separate from validate.yml): - detect-harness: hermetic run-detect-tests.sh - validate-harness: 5 toolchain setups + yq, run-tests.sh (must reach exit 0) - detect -> consume: trimmed needs.*.outputs isolation proof (non-empty case; docs-only empty-result contract already pinned by run-detect-tests.sh case h) Repoint now-dangling references to the deleted workflows in CLASSIFICATION.md and detect-changed-samples.README.md; add a post-consolidation banner to the historical P1.4 WRAPPER-MAPPING.md audit. Validated: actionlint + shellcheck clean; hermetic detect harness 23/23 GREEN locally; validate harness PARTIAL locally with 0 failures (toolchain-gated cases skip, provided by the runner).
Contributor
|
👋 Thanks for your contribution, Brandon Miller (@brandom-msft)! This repository is read-only. If you are contributing on behalf of Microsoft, please submit your PR to the private staging repository instead: See CONTRIBUTING.md for full instructions. |
Brandon Miller (brandom-msft)
temporarily deployed
to
L4-validation
July 30, 2026 06:43 — with
GitHub Actions
Inactive
This branch was previously deployed
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.
Consolidates the two inert *-selftest.yml dev harnesses into one lean, real-triggered
.github/workflows/scripts-selftest.ymlthat runs the script regression harnesses on any PR touching.github/scripts/**.Jobs (
contents: readonly, non-required, separate fromvalidate.yml):detect-harness— hermeticrun-detect-tests.shvalidate-harness— 5 toolchain setups + yq,run-tests.sh(must reach exit 0)detect->consume— trimmedneeds.*.outputsisolation proof (non-empty case)Also deletes the two old workflows and repoints now-dangling doc references.
Stacked on
brandom-msft-p2-2-trusted-l4-lane. Opened primarily as a real-runner green-proof probe for the new workflow; the redirect bot will auto-close it, but the triggered checks run to completion.