[ci] Read docs.rs arguments from Cargo metadata - #3585
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gx6vwydb7e2ijorcpxmqt3axxviika37l #3585 +/- ##
====================================================================
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:
|
79fdfe7 to
ecdf5a5
Compare
0a6b895 to
4797a98
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. Delightful! 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". |
ecdf5a5 to
7fa05bc
Compare
4797a98 to
9f4245d
Compare
a68ba4a to
e9352c2
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". |
e9352c2 to
bec75d1
Compare
23c7af1 to
9940ac1
Compare
|
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". |
bec75d1 to
eb2e036
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. 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". |
eb2e036 to
e663714
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
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". |
3073096 to
f7b052c
Compare
e663714 to
10de534
Compare
f7b052c to
fe6e3ac
Compare
10de534 to
517c46a
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
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". |
fe6e3ac to
bee291c
Compare
1f30180 to
58c3a2a
Compare
bee291c to
5ea2766
Compare
The nightly documentation command must include the exact Rustdoc
arguments that docs.rs uses. Keeping another list in the command model
would let the two copies drift.
Read the ordered arguments from the canonical Zerocopy package
selected by CI policy. Reject missing or malformed metadata, ownership
by the wrong package, and values whose argument boundaries the current
RUSTDOCFLAGS transport cannot preserve. Expose only checked arguments
to later planning code.
Tests: cargo test -p zc --all-targets
cargo clippy -p zc --all-targets --offline -- -D warnings
*Authored by an agent, posting via joshlf's account*
gherrit-pr-id: Getgur4i3su6l2xcpgewl2bbvcskszm5r
5ea2766 to
55b1919
Compare
58c3a2a to
223dfa4
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head, |
|
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". |
2 participants
The nightly documentation command must include the exact Rustdoc
arguments that docs.rs uses. Keeping another list in the command model
would let the two copies drift.
Read the ordered arguments from the canonical Zerocopy package
selected by CI policy. Reject missing or malformed metadata, ownership
by the wrong package, and values whose argument boundaries the current
RUSTDOCFLAGS transport cannot preserve. Expose only checked arguments
to later planning code.
Tests: cargo test -p zc --all-targets
cargo clippy -p zc --all-targets --offline -- -D warnings
Authored by an agent, posting via joshlf's account
Latest Update: v29 — Compare vs v28
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Getgur4i3su6l2xcpgewl2bbvcskszm5r && git checkout -b pr-Getgur4i3su6l2xcpgewl2bbvcskszm5r FETCH_HEADCheckout
git fetch origin refs/heads/Getgur4i3su6l2xcpgewl2bbvcskszm5r && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Getgur4i3su6l2xcpgewl2bbvcskszm5r && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.