[ci] Share typed toolchain metadata - #3571
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## Gvc5xhb5dgtwxc6zn6aycox7svxcbv3by #3571 +/- ##
==================================================================
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:
|
1355162 to
6ed1263
Compare
1bae4b8 to
b76c6da
Compare
b76c6da to
d6eb90c
Compare
6ed1263 to
53a0bf8
Compare
|
Authored by an agent, posting via joshlf's account @codex review Please review the current head. Treat any earlier review as stale if it targets |
|
Codex Review: Didn't find any major issues. You're on a roll. 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". |
d6eb90c to
da31221
Compare
53a0bf8 to
765e70e
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". |
765e70e to
896e57f
Compare
da31221 to
38d4762
Compare
38d4762 to
91a3471
Compare
896e57f to
7e0d1ce
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". |
7e0d1ce to
bbc3d09
Compare
91a3471 to
0b6d622
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". |
Add the zc repository-tool library with a typed, path-aware reader for Zerocopy's MSRV, pinned toolchains, and build-script version cfgs. Use the shared reader in cargo-zerocopy without changing its command-line or process behavior. Parse the manifest directly because this bootstrap path must choose a toolchain before it can safely invoke Cargo. Document the separate textual Cargo.toml contract which build.rs consumes. This prevents future metadata work from treating TOML validity as enough. Normalize the two edited tool manifests from their legacy CRLF form to LF. This keeps their new lines from appearing as trailing whitespace and gives future edits one consistent representation. Tests: ./ci/check_tools.sh; focused rustfmt; cargo-zerocopy version lookups. *Authored by an agent, posting via joshlf's account* gherrit-pr-id: Gga2afs3rc7lmrmvkzm4wdcmi4lwlyi65
0b6d622 to
ceedb57
Compare
bbc3d09 to
6b8dd79
Compare
2 participants
Add the zc repository-tool library with a typed, path-aware reader for
Zerocopy's MSRV, pinned toolchains, and build-script version cfgs.
Use the shared reader in cargo-zerocopy without changing its
command-line or process behavior. Parse the manifest directly because
this bootstrap path must choose a toolchain before it can safely invoke
Cargo.
Document the separate textual Cargo.toml contract which build.rs
consumes. This prevents future metadata work from treating TOML validity
as enough.
Normalize the two edited tool manifests from their legacy CRLF form to
LF. This keeps their new lines from appearing as trailing whitespace and
gives future edits one consistent representation.
Tests: ./ci/check_tools.sh; focused rustfmt; cargo-zerocopy version
lookups.
Authored by an agent, posting via joshlf's account
Latest Update: v13 — Compare vs v12
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Gga2afs3rc7lmrmvkzm4wdcmi4lwlyi65 && git checkout -b pr-Gga2afs3rc7lmrmvkzm4wdcmi4lwlyi65 FETCH_HEADCheckout
git fetch origin refs/heads/Gga2afs3rc7lmrmvkzm4wdcmi4lwlyi65 && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Gga2afs3rc7lmrmvkzm4wdcmi4lwlyi65 && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.