Repository navigation
chore(deps): bump actions/cache from 4.3.0 to 6.1.0 - #544
dependabot[bot] wants to merge 7 commits into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Castiron custom codeEvaluated main: Trusted main policy: ✅ No new custom-code files detected. 0 mixed files remain; 0 existing customizations changed. Compared A changed generated baseline means this report cannot reliably identify which handwritten lines changed. Inspect the custom-code diffDownload the exact patch produced by this run (requires repository access): gh run download 39464674413 --repo openai/openai-cli \
--name castiron-custom-code-39464674413-1 --dir /tmp/castiron-custom-code-39464674413-1
git apply --stat /tmp/castiron-custom-code-39464674413-1/custom-code.patch
cat /tmp/castiron-custom-code-39464674413-1/custom-code.patchOr reproduce it from an SDK checkout containing the vendored reporter: git fetch --no-tags origin 04e59e00fcceb6ad1145f0b739448403127c1c2c 77c9f95032b420650d88ac23132c4cec7b6192d8
python3 scripts/castiron/custom_code_report.py report \
--base 04e59e00fcceb6ad1145f0b739448403127c1c2c \
--head 77c9f95032b420650d88ac23132c4cec7b6192d8 --fetch --require-head-hash --public \
--out /tmp/castiron-custom-code-77c9f95032b4
cat /tmp/castiron-custom-code-77c9f95032b4/custom-code.patchThis is the current full custom patch for mixed files, not an attribution of only the handwritten lines changed by this PR. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2bab2ee6c4
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| echo "modules=$(go env GOMODCACHE)" >> "$GITHUB_OUTPUT" | ||
| - name: Cache artifact build dependencies and compilation | ||
| uses: actions/cache@0057852bfaa89a56745cba8c7296529d2fc39830 # v4.3.0 | ||
| uses: actions/cache@55cc8345863c7cc4c66a329aec7e433d2d1c52a9 # v6.1.0 |
There was a problem hiding this comment.
Update the cache-pin regression assertion
The repository test TestArtifactBuildCacheIsSeparateAndRefreshable in internal/releaseworkflow/workflow_test.go:960-962 still requires the previous 0057852... SHA, so every test run containing this package will fail after the workflow starts using 55cc834.... Update that assertion alongside the action bump so the structural validation accepts the reviewed v6.1.0 pin.
AGENTS.md reference: AGENTS.md:L81-L86
Useful? React with 👍 / 👎.
|
[saibot] before: artifact builds used an older cache action, without the newer read-only cache diagnostics. now: this updates the pinned action, but the repository test still requires the previous pin. verdict: request changes, useful maintenance, but update the cache-pin assertion before merging. the mismatch fails repository tests and blocks the required test check. |
a53d829 to
2b9317f
Compare
Bumps [actions/cache](https://github.com/actions/cache) from 4.3.0 to 6.1.0. - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@0057852...55cc834) --- updated-dependencies: - dependency-name: actions/cache dependency-version: 6.1.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
2b9317f to
6e038ac
Compare
Bumps actions/cache from 4.3.0 to 6.1.0.
Release notes
Sourced from actions/cache's releases.
... (truncated)
Changelog
Sourced from actions/cache's changelog.
... (truncated)
Commits
55cc834Merge pull request #1768 from jasongin/readonly-cached8cd72fBump@actions/cacheto v6.1.0 - handle cache write error due to RO token2c8a9bdMerge pull request #1760 from actions/samirat/esm_migration_and_package_updatee9b91fdPrettier fixese4884b8Rebuild dist10baf01Fixed licensese39b386Fix test mock return orderb692820PR feedback6074912Rebuild dist bundles as ESM to match type:module5a912e8Fix lint and jest issues