-
Notifications
You must be signed in to change notification settings - Fork 217
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
Allow disabling cache in setup-r-dependencies
#530
Comments
Do you want to submit a PR? |
I'm really confused because I was thinking https://github.com/Bisaloo/actions/commit/f4a22e74b7856a66b20aa93365141a2fbb80810d would be enough but it doesn't look like it's working 🤔 gh::gh("/repos/{owner}/{repo}/actions/cache/usage", owner = "Bisaloo", repo = "testpkg")
#> {
#> "full_name": "Bisaloo/testpkg",
#> "active_caches_size_in_bytes": 4860071,
#> "active_caches_count": 1
#> } Created on 2022-03-23 by the reprex package (v2.0.1.9000) https://github.com/Bisaloo/testpkg/blob/main/.github/workflows/R-CMD-check.yaml |
@Bisaloo, I think you need to do use comparison, see e.g. Lines 19 to 27 in 1efdaa6
Also note you do not need |
Ah yes, thanks, that seems to have done the trick! It's quite strange you cannot use boolean values directly |
This issue has been automatically locked. If you believe you have found a related problem, please file a new issue and include a link to this issue |
Is your feature request related to a problem? Please describe.
I'm trying to use
act
to test workflows locally and avoid spamming commits when I'm developing a complex workflow. Unfortunately, it looks likeact
doesn't support caching and fails whenever caching is included in a step.Describe the solution you'd like
I would like to have a way to disable the cache in
setup-r-dependencies
. Then, it would be possible foract
users to do:Describe alternatives you've considered
I can't think of any other good alternative. The only option would be to go way to the 'old' way of manually installing dependencies (with remotes) but it means losing all the benefit brought by
setup-r-dependencies
.The text was updated successfully, but these errors were encountered: