Skip to content

Bump @actions/cache to 5.1.0, log cache write denied#1324

Open
jasongin wants to merge 4 commits into
actions:mainfrom
jasongin:update-actions-cache-5.1.0
Open

Bump @actions/cache to 5.1.0, log cache write denied#1324
jasongin wants to merge 4 commits into
actions:mainfrom
jasongin:update-actions-cache-5.1.0

Conversation

@jasongin

@jasongin jasongin commented Jun 18, 2026

Copy link
Copy Markdown

Description:

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Copilot AI review requested due to automatic review settings June 18, 2026 22:28
@jasongin jasongin requested a review from a team as a code owner June 18, 2026 22:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/cache to ^5.1.0 and bump action version to 6.3.0.
  • Add a debug log when saveCache returns -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

Comment thread src/cache-save.ts Outdated
Comment thread src/cache-save.ts Outdated
@jasongin

Copy link
Copy Markdown
Author

The e2e-cache and Validate PyPy e2e workflows are unrelated to this PR; they are failing on main branch also. It looks like they're fixed by #1322.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants