[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". |
*Authored by Codex, posting via joshlf's account* 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. Tests: ./ci/check_tools.sh; focused rustfmt; cargo-zerocopy version lookups. gherrit-pr-id: Gga2afs3rc7lmrmvkzm4wdcmi4lwlyi65
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". |
2 participants
Authored by Codex, posting via joshlf's account
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.
Tests: ./ci/check_tools.sh; focused rustfmt; cargo-zerocopy version
lookups.
Latest Update: v4 — Compare vs v3
📚 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.