[ci] Make pre-push checks safe to run in parallel - #3574
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3574 +/- ##
=======================================
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 Please review the current head. Treat any earlier review as stale if it targets |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: df82221d35
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
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".
df82221 to
c08a2eb
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". |
c08a2eb to
73337ca
Compare
73337ca to
b0e9d01
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
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 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". |
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? 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". |
Fresh runners can start several cargo-zerocopy processes before rustup has installed the pinned nightly toolchain. Rustup shares download state between those processes, so concurrent installation can fail when one process moves another process's partial file. Install nightly before starting the checks in parallel. Preserve the hook protocol on stdin, scope automatic installation to the bootstrap command, and wait for every child. An early failure no longer discards diagnostics from checks that are still running. Also snapshot every first-party Cargo.lock before running the nominally read-only checks and reject any change. A hermetic fake-repository test covers bootstrap ordering, stdin preservation, lockfile accounting, subdirectory invocation, and child failures. *Authored by an agent, posting via joshlf's account* gherrit-pr-id: Gdqabzhpevkhxg3fzk6bvbtpws7nk2vwv
b0e9d01 to
aa10436
Compare
|
Authored by an agent, posting via joshlf's account @codex review |
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. 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
Fresh runners can start several cargo-zerocopy processes before rustup
has installed the pinned nightly toolchain. Rustup shares download state
between those processes, so concurrent installation can fail when one
process moves another process's partial file.
Install nightly before starting the checks in parallel. Preserve the
hook protocol on stdin, scope automatic installation to the bootstrap
command, and wait for every child. An early failure no longer discards
diagnostics from checks that are still running.
Also snapshot every first-party Cargo.lock before running the nominally
read-only checks and reject any change. A hermetic fake-repository test
covers bootstrap ordering, stdin preservation, lockfile accounting,
subdirectory invocation, and child failures.
Authored by an agent, posting via joshlf's account
Latest Update: v20 — Compare vs v19
📚 Full Patch History
Links show the diff between the row version and the column version.
⬇️ Download this PR
Branch
git fetch origin refs/heads/Gdqabzhpevkhxg3fzk6bvbtpws7nk2vwv && git checkout -b pr-Gdqabzhpevkhxg3fzk6bvbtpws7nk2vwv FETCH_HEADCheckout
git fetch origin refs/heads/Gdqabzhpevkhxg3fzk6bvbtpws7nk2vwv && git checkout FETCH_HEADCherry Pick
git fetch origin refs/heads/Gdqabzhpevkhxg3fzk6bvbtpws7nk2vwv && git cherry-pick FETCH_HEADPull
Stacked PRs enabled by GHerrit.