-
-
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
chore: modify dependency caching on bitrise #8561
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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8561 +/- ##
=======================================
Coverage 40.90% 40.90%
=======================================
Files 1245 1245
Lines 30150 30150
Branches 2895 2895
=======================================
Hits 12333 12333
Misses 17114 17114
Partials 703 703 ☔ View full report in Codecov by Sentry. |
node_modules
and yarn cache
from bitrise
node_modules
and yarn cache
from bitrisenode_modules
and yarn cache
from bitrise
node_modules
and yarn cache
from bitrise
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/e34d0221-adf4-4042-89da-784a2e72feba |
|
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/5b36676b-cb2e-41ce-8ed6-e7ad549a998e |
Description
yarn install
to reduce bitrise build times by approximately 1m46s.It is a "quick win" until a decision is made on a caching strategy.
Passing
pr_smoke_e2e_pipeline
Passing
pr_regression_e2e_pipeline
Background and discussion in this internal document, but the gist is:
node_modules
.yarn install
.code_setup
workflow duration decreases from approximately 5m29s to 3m53m.Related issues
Fixes: https://github.com/MetaMask/mobile-planning/issues/1512
Pre-merge author checklist
Pre-merge reviewer checklist