Fix caching of target
dir for fuzz
project
#96
Merged
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.
Currently caching does not work for
fuzz
job which can be verified by looking at Github workflow outputs.The main cause of the issue is the config
workspaces: fuzz -> target
. The fuzz target does not use a proper workspace but a separate crate. There is a pending issue discussing that.The
rust-cache
action is executed in the root of the project and is not able to locatefuzz
directory by workspace name.I decided to list the directories to cache instead of relying on target dir detection which seems to work.
I also fixed the path
fuzz/corpus
toi18n-helpers/fuzz/corpus
.I adjusted the caching condition
save-if
to cache only onmain
builds. I think this makes sense because we don't need to cache PRs which might not get merged or get updated multiple times during review.