Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to cache@v4 #71

Merged
merged 1 commit into from
Jun 2, 2024
Merged

Conversation

jan-kiszka
Copy link
Contributor

Just a trivial drive-by fix while playing with this.

Fixes "Node.js 16 actions are deprecated" warning.

Signed-off-by: Jan Kiszka <[email protected]>
@ArneTR
Copy link
Member

ArneTR commented Jun 2, 2024

Hey @jan-kiszka

thanks for the PR, always welcome! ❤️

It makes total sense to update the action to cache@v4. Since you call it a "fix" though I thought I inquire: Have you seen any difference to v3 in an metric or behaviour?

From the [Release Notes]8https://github.com/actions/cache/releases) I see that only node was updated and not much more.

@ArneTR ArneTR merged commit fa28478 into green-coding-solutions:main Jun 2, 2024
2 checks passed
@jan-kiszka
Copy link
Contributor Author

The "fix" is more related to the cited warnings shown for the CI run using it.

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.

2 participants