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

Use Julia-actions/cache uniformly #53

Closed
suvayu opened this issue Apr 18, 2024 · 3 comments · Fixed by #162
Closed

Use Julia-actions/cache uniformly #53

suvayu opened this issue Apr 18, 2024 · 3 comments · Fixed by #162
Assignees
Labels
template: change Related to changes to the template, not the package - for new stuff, use 'addition to template'

Comments

@suvayu
Copy link
Collaborator

suvayu commented Apr 18, 2024

I'm getting these in CI:

[julia-buildpkg] Caching of the julia depot was not detected
Consider using julia-actions/cache to speed up runs https://github.com/julia-actions/cache. To ignore, set input ignore-no-cache: true

I guess now we don't need to rummage through docs how to do it correctly?

@abelsiqueira abelsiqueira added the template: change Related to changes to the template, not the package - for new stuff, use 'addition to template' label May 1, 2024
@abelsiqueira abelsiqueira added this to the 0.4.0 milestone May 15, 2024
@abelsiqueira
Copy link
Collaborator

Julia-actions/cache uses the actions/cache internally. Sometimes we use one or the other or none. We should change to use Julia-actions/cache in all places. I will update the title.

@abelsiqueira abelsiqueira changed the title Speed-up CI with caching Use Julia-actions/cache uniformly May 21, 2024
@abelsiqueira
Copy link
Collaborator

@allcontributors please add @suvayu for bug

Copy link
Contributor

@abelsiqueira

I've put up a pull request to add @suvayu! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
template: change Related to changes to the template, not the package - for new stuff, use 'addition to template'
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants