Bump @actions/cache to 5.1.0, log cache write denied#1324
Open
jasongin wants to merge 4 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request updates the action’s caching dependency (@actions/cache) to pick up improved handling/logging for cache write denials (e.g., read-only tokens), and aligns the action’s own cache-save logging and published artifacts with that behavior.
Changes:
- Bump
@actions/cacheto^5.1.0and bump action version to6.3.0. - Add a debug log when
saveCachereturns-1(cache not saved) to avoid redundant warnings while still providing traceability. - Update generated
dist/**bundles and dependency license metadata to reflect the dependency update.
Show a summary per file
| File | Description |
|---|---|
| src/cache-save.ts | Adds debug logging when cache save returns -1 (not saved). |
| package.json | Bumps action version to 6.3.0 and @actions/cache to ^5.1.0. |
| package-lock.json | Updates lockfile entries for the version bump and cache dependency. |
| dist/setup/index.js | Updates bundled @actions/cache code to include cache write denied handling. |
| dist/cache-save/index.js | Updates bundled output, including the new -1 debug log and cache write denied handling. |
| .licenses/npm/@actions/cache.dep.yml | Updates dependency license metadata for @actions/cache 5.1.0. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 3/6 changed files
- Comments generated: 2
Author
|
The e2e-cache and Validate PyPy e2e workflows are unrelated to this PR; they are failing on |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
@actions/cacheto v5.1.0 to pick up actions/toolkit#2435 Handle cache write error due to read-only tokendistCheck list: