[ci] Validate all inputs before CI planning - #3579
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gocqawbn3p3zj7xyfo7etk6vo4633r3tn #3579 +/- ##
====================================================================
Coverage ? 91.85%
====================================================================
Files ? 20
Lines ? 6093
Branches ? 0
====================================================================
Hits ? 5597
Misses ? 496
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
5070ce3 to
fac6e43
Compare
bdc8039 to
16d8463
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
16d8463 to
aeefea6
Compare
fac6e43 to
77f85fe
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Add one loading boundary which reads the typed policy, audits live Cargo metadata and repository files, inventories every workflow job, and strictly parses the frozen legacy baseline. Return planning inputs only after every layer succeeds. Wiring the workflow audit here also removes the temporary unused-code allowance from its introductory commit. This keeps future planners from depending on call order or accidentally skipping a validation pass. It also keeps the policy-owned baseline paths in one place rather than repeating their coordination in each consumer. Tests: CARGO_NET_OFFLINE=true ./ci/check_tools.sh Tests: ./ci/check_fmt.sh *Authored by an agent, posting via joshlf's account* gherrit-pr-id: Gsizgstaatljzyuwyc4tp4ha6krvibz64
77f85fe to
fded6e0
Compare
aeefea6 to
a008f12
Compare
2 participants
Add one loading boundary which reads the typed policy, audits live Cargo
metadata and repository files, inventories every workflow job, and
strictly parses the frozen legacy baseline. Return planning inputs only
after every layer succeeds. Wiring the workflow audit here also removes
the temporary unused-code allowance from its introductory commit.
This keeps future planners from depending on call order or accidentally
skipping a validation pass. It also keeps the policy-owned baseline
paths in one place rather than repeating their coordination in each
consumer.
Tests: CARGO_NET_OFFLINE=true ./ci/check_tools.sh
Tests: ./ci/check_fmt.sh
Authored by an agent, posting via joshlf's account
Latest Update: v9 — Compare vs v8
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Gsizgstaatljzyuwyc4tp4ha6krvibz64 && git checkout -b pr-Gsizgstaatljzyuwyc4tp4ha6krvibz64 FETCH_HEADCheckout
git fetch origin refs/heads/Gsizgstaatljzyuwyc4tp4ha6krvibz64 && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Gsizgstaatljzyuwyc4tp4ha6krvibz64 && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.