[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%
====================================================================
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:
|
|
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
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". |
364653d to
e28ec36
Compare
f27177f to
d4a566d
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! 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". |
e28ec36 to
ee675b9
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Keep it up! 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". |
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
ee675b9 to
c370bbe
Compare
8cb3a1e to
7fd0482
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Nice work! 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: v14 — Compare vs v13
📚 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.