Bump @actions/cache to 5.1.0, log cache write denied#1569
Open
jasongin wants to merge 3 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates setup-node’s caching behavior to align with @actions/cache@5.1.0, including improved handling/logging when cache writes are denied (for example, read-only tokens on fork PRs). It also bumps the action’s minor version and refreshes generated artifacts and dependency licensing metadata.
Changes:
- Bump
@actions/cachedependency to^5.1.0and update action version to6.5.0. - Add a debug log when
saveCachereturns-1(cache not saved) and update tests accordingly. - Regenerate
distbundles and update the cached dependency license file.
Show a summary per file
| File | Description |
|---|---|
src/cache-save.ts |
Adds a debug log when @actions/cache.saveCache indicates the cache was not saved (-1). |
__tests__/cache-save.test.ts |
Updates test expectations to validate the new debug log behavior. |
package.json |
Bumps action version to 6.5.0 and updates @actions/cache to ^5.1.0. |
package-lock.json |
Locks @actions/cache to 5.1.0 and updates the root version metadata. |
dist/setup/index.js |
Regenerated bundle reflecting the updated @actions/cache behavior. |
dist/cache-save/index.js |
Regenerated bundle reflecting the new cache-write-denied handling and the new debug log. |
.licenses/npm/@actions/cache.dep.yml |
Updates recorded dependency version for @actions/cache to 5.1.0. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 4/7 changed files
- Comments generated: 1
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: