-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix: Fix/9345 bitrise cache failure #9611
Conversation
This reverts commit 57f3323.
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good 🙏
|
Description
As suggested by the Bitrise team, the changes in the PR replaces both the
save
andrestore
step withdeploy
andpull
, which shares intermediate files within pipelines instead of caching on a global level. An extra step added postpull
step was to copy the pulled files into the directories that Detox uses to find the builds. Lastly, a small update was made to thebrew install ccache
command to include the flagHOMEBREW_NO_DEPENDENTS_CHECK=1
, which should skip updating packages that depend onccache
, therefore speeding up the step.Related issues
Fixes: #9345
Manual testing steps
pr_smoke_e2e_pipeline
and ensure that both iOS and Android are building and testing successfullyScreenshots/Recordings
N/A
Before
N/A
After
N/A
Pre-merge author checklist
Pre-merge reviewer checklist