feat(cache): change key to match actions/cache documentation#213
feat(cache): change key to match actions/cache documentation#213mmorel-35 wants to merge 1 commit intoactions:mainactions/setup-java:mainfrom mmorel-35:mainCopy head branch name to clipboard
Conversation
It is caused by the difference between LF (Ubuntu) and CRLF (Windows). I also faced this problem before. |
|
Note about the context of current key naming:
|
Signed-off-by: Matthieu MOREL <mmorel-35@users.noreply.github.com>
|
Hi @KengoTODA, I found another way , to correct the file, I'm creating a workflow where I execute the build and then have a task to commit the changed file directly on my repository, see here for example Concerning the notion of prefix I'm not sure to really understand the reason behind that. Does it means that the provided examples in actions/cache are wrong ? Their is no explanation behind that change in the comment. |
It's not wrong. And there is no strict official rule to design the cache key... what we need is just grab how actions/cache matches a cache key and design optimized key and restore keys. So I just shared the context of current change... repository owner will judge how the key should be designed. |
Description:
This PR change the cache key construction to fit those examples. It also add the use of restoreKeys and a more verbose log when retrieving fails.
It also add the pre-checkin npm task as it exists on the other setup-X
I don't see how to fix this but the "Verify no unstaged changes" is failing. I'm working on Windows but usually it doesn't matter.
Any idea ?
Related issue:
N/A
Check list: