Skip to content

ci: parameterize Cargo target paths - #31332

Merged
anp-oai merged 3 commits into
mainfrom
codex/dev-drive-setup-ci
Jul 8, 2026
Merged

ci: parameterize Cargo target paths#31332
anp-oai merged 3 commits into
mainfrom
codex/dev-drive-setup-ci

Conversation

@anp-oai

@anp-oai anp-oai commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Why

Prepare CI jobs for a later build-output relocation without changing where they write today.

What

  • Export CARGO_TARGET_DIR from setup-ci at the existing codex-rs/target path.
  • Route nextest, release, artifact, and signing paths through CARGO_TARGET_DIR.
  • Require V8 staging callers to pass an explicit target directory while preserving the existing upstream path.

Manual validation

  • Ran just test-github-scripts.
  • Parsed GitHub Actions YAML with yq.

Stack

  • #31332 — parameterize Cargo target paths
  • #31356 — Windows 2025 runner bump
  • #31357 — Dev Drive I/O routing

anp-oai added a commit that referenced this pull request Jul 7, 2026
anp-oai added a commit that referenced this pull request Jul 7, 2026
@anp-oai
anp-oai force-pushed the codex/dev-drive-setup-ci branch from 4fe566a to c081cf5 Compare July 7, 2026 04:19
@anp-oai anp-oai changed the title ci: route build IO through shared setup ci: parameterize Cargo target paths Jul 7, 2026
Comment thread .github/scripts/rusty_v8_bazel.py Outdated
Comment thread .github/workflows/rust-ci-full-nextest-platform.yml Outdated
anp-oai added a commit that referenced this pull request Jul 7, 2026
@anp-oai
anp-oai marked this pull request as ready for review July 7, 2026 06:09
@anp-oai

anp-oai commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: efdc44238a

ℹ️ 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".

@anp-oai

anp-oai commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

@codex review

anp-oai added a commit that referenced this pull request Jul 7, 2026
@anp-oai
anp-oai force-pushed the codex/dev-drive-setup-ci branch from c769674 to 707ce5c Compare July 7, 2026 18:42
@anp-oai
anp-oai force-pushed the codex/dev-drive-setup-ci branch from 707ce5c to 329f08d Compare July 7, 2026 23:54
@anp-oai
anp-oai enabled auto-merge (squash) July 7, 2026 23:54
@anp-oai
anp-oai merged commit 77b766c into main Jul 8, 2026
47 checks passed
@anp-oai
anp-oai deleted the codex/dev-drive-setup-ci branch July 8, 2026 00:02
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants