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

Add example using GHA cache backend #12

Merged
merged 9 commits into from
May 16, 2024
Merged

Add example using GHA cache backend #12

merged 9 commits into from
May 16, 2024

Conversation

riisi
Copy link
Contributor

@riisi riisi commented Oct 31, 2023

Raising for info at this stage pending 2.19 release.

Note that no changes are required to run this locally - i.e. it will not fail due to missing GH tokens but fall back to local cache if present.

I think I will also attempt to add an example using the Inline cache backend, publishing to Github packages - this seems a better option when there are many images as the GHA cache limit would be reached.

Copy link
Contributor

@benjyw benjyw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool! I'll ask @kaos to take a quick look as well.

@benjyw benjyw requested a review from kaos November 6, 2023 02:05
Copy link
Member

@kaos kaos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm.

I'm not familiar with the values for cache_from and cache_to in particular, but trust they do what is intended ;)

@benjyw
Copy link
Contributor

benjyw commented Nov 9, 2023

@riisi this is still a Draft. Should it be?

@riisi
Copy link
Contributor Author

riisi commented Nov 10, 2023 via email

@huonw
Copy link
Contributor

huonw commented Mar 25, 2024

2.19 is released. Is this ready to go again?

@riisi riisi marked this pull request as ready for review March 26, 2024 22:39
@riisi
Copy link
Contributor Author

riisi commented Mar 26, 2024

I think this should be good to go now.

@huonw huonw merged commit ec47248 into pantsbuild:main May 16, 2024
2 checks passed
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.

4 participants