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

[ci] --ci option enables some caching #242

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

michaelkirk
Copy link
Member

@michaelkirk michaelkirk commented Jan 31, 2023

Hopefully this speeds up our earthly builds.

@michaelkirk
Copy link
Member Author

I don't see any obvious improvement in build time (or really any change in behavior at all) while using this flag.

I feel like I'm misunderstanding something. 😞

@ellenhp
Copy link
Member

ellenhp commented Feb 5, 2023

I think you might need to add a cache action to the CI config.

The "push latest" jobs will write to the cache, everything else will
only read from the cache.

TODO: currently the dev push job also writes to the cache. This is

for testing - I think probably only the push-latest job should write

the cache to avoid thrashing the cache.
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.

None yet

2 participants