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

chore: don't create docker cache on PRs #3059

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

stuartwdouglas
Copy link
Contributor

No description provided.

@stuartwdouglas stuartwdouglas requested review from alecthomas and a team as code owners October 9, 2024 21:29
@stuartwdouglas stuartwdouglas requested review from gak and removed request for a team October 9, 2024 21:29
@stuartwdouglas stuartwdouglas force-pushed the stuartwdouglas/dont-create-docker-pr-cache branch from 4c7cd82 to 9463599 Compare October 9, 2024 21:30
This was referenced Oct 9, 2024
@alecthomas
Copy link
Collaborator

ci.yml doesn't currently run on main, so we might want to either do that or explicitly add the Docker caching to writecache.yml

- Don't create docker cache on PR
- Change the way maven write caching works
@stuartwdouglas stuartwdouglas force-pushed the stuartwdouglas/dont-create-docker-pr-cache branch from 9463599 to 8fc8e6c Compare October 9, 2024 21:39
@stuartwdouglas stuartwdouglas merged commit f34de9d into main Oct 10, 2024
95 checks passed
@stuartwdouglas stuartwdouglas deleted the stuartwdouglas/dont-create-docker-pr-cache branch October 10, 2024 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants