-
-
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: cocoapod cache issue #8306
Conversation
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. |
e7929e0
to
ec905e1
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8306 +/- ##
=======================================
Coverage 40.20% 40.20%
=======================================
Files 1235 1235
Lines 29926 29926
Branches 2872 2872
=======================================
Hits 12033 12033
Misses 17200 17200
Partials 693 693 ☔ View full report in Codecov by Sentry. |
ec905e1
to
848b409
Compare
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/f0516007-124a-4beb-ba40-c8305c669c45 |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/7264e209-ab57-41db-bf18-9d4b7f541d41 |
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.
LGTM
|
Description
This PR aims to fix an issue when run ios build when a dependency is updated. After analysing the latest cases reported facing issues with ios build I realised that cocoapod restore was overwriting the cocoa pod install in the
yarn setup
step. So, I moved the restore cocoa pod cache on top of theyarn setup
step to take advantage of the cache and properly update the new dependencies.Pipeline error: https://app.bitrise.io/build/8ae57475-c72a-41ce-96f2-d3b6768b9d16
Error:
Related issues
Fixes: https://github.com/MetaMask/mobile-planning/issues/1487
Manual testing steps
Screenshots/Recordings
Before
Pipeline updating cocoa pod:
https://app.bitrise.io/build/8ae57475-c72a-41ce-96f2-d3b6768b9d16
After
Pipeline updating cocoa pod adding back the cocopod install step:
https://app.bitrise.io/build/8631a2cf-89f7-4025-964f-4c8bfeb4e28c
Pre-merge author checklist
Pre-merge reviewer checklist