-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Make save/restore logs akin #509
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
Conversation
@yacaovsnc @konradpabjan Here is one more change necessary for closing #471, but I couldn't manage how to update the toolkit dependency. Could you guide me, please? |
@rosik we need to bump up the version in the toolkit and then publish it so the changes are here: https://www.npmjs.com/package/@actions/cache I'll take care of publishing a new version this week. After that you update |
@konradpabjan ok, thanks. |
Apologies for the delay. I've published a new version of |
I've updated Could you resolve the merge conflicts in your PR? We can get the other changes in |
- Print primary key when saving cache
@konradpabjan done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Thanks again for all the contributions and sorry for the delays on our side. I'm going to wait until tomorrow before updating any tags or creating a new release ( last-minute sanity check after larger changes go in). Quite a bit of users use |
This is a follow-up for actions/toolkit#650
Close #471