[ci] Project only executable cell selectors - #3590
Conversation
97e206f to
5527487
Compare
1c25593 to
1632fd7
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gbsj7gxuv5hpi22zhup5w3fajtjjxnldj #3590 +/- ##
==================================================================
Coverage 91.85% 91.85%
==================================================================
Files 20 20
Lines 6093 6093
==================================================================
Hits 5597 5597
Misses 496 496 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Bravo. 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". |
1632fd7 to
ed2ac06
Compare
ed2ac06 to
acbdda5
Compare
7bb8725 to
00059a2
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". |
acbdda5 to
b163bc5
Compare
00059a2 to
4044dad
Compare
The compact Actions matrices still duplicated derived feature arguments,
target behavior, and Miri flags. The typed executor deliberately
resolves those details again from checked policy, so carrying both
representations would create a second contract which could drift.
Emit only the complete selectors accepted by each executor. Keep the
richer semantics in the human-facing plan artifact, where they remain
useful for review without becoming execution input.
Add exact compact-schema and output-size assertions so future transport
growth requires deliberate review.
Tests: cargo +1.93.1 test -p zc --all-targets --offline
cargo +1.93.1 clippy -p zc --all-targets --offline --
-D warnings
*Authored by an agent, posting via joshlf's account*
gherrit-pr-id: Gkr4fd4vz5vegygalblsmjjgpdq3eulsx
4044dad to
f27177f
Compare
b163bc5 to
364653d
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. 👍 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". |
2 participants
The compact Actions matrices still duplicated derived feature arguments,
target behavior, and Miri flags. The typed executor deliberately
resolves those details again from checked policy, so carrying both
representations would create a second contract which could drift.
Emit only the complete selectors accepted by each executor. Keep the
richer semantics in the human-facing plan artifact, where they remain
useful for review without becoming execution input.
Add exact compact-schema and output-size assertions so future transport
growth requires deliberate review.
Tests: cargo +1.93.1 test -p zc --all-targets --offline
cargo +1.93.1 clippy -p zc --all-targets --offline --
-D warnings
Authored by an agent, posting via joshlf's account
Latest Update: v10 — Compare vs v9
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Gkr4fd4vz5vegygalblsmjjgpdq3eulsx && git checkout -b pr-Gkr4fd4vz5vegygalblsmjjgpdq3eulsx FETCH_HEADCheckout
git fetch origin refs/heads/Gkr4fd4vz5vegygalblsmjjgpdq3eulsx && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Gkr4fd4vz5vegygalblsmjjgpdq3eulsx && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.