Improve CI Efficiency - #1625
Conversation
…ncy resolution step
Pull Request Test Coverage Report for Build fdb21652-b67b-4323-bcf7-7fa63a2b819bDetails
💛 - Coveralls |
|
Hi David, in this PR I've added partial dependency caching as well as parallelism during the build process after a cache miss. However, I'm not sure how we can (or if we can) test these changes without actually merging the PR and seeing it work in production. Since the changes would require either a partial or full cache miss to test (which is not happening with this PR), I wanted to ask for your feedback before proceeding further. Specifically, with my parallel jobs change, I looked here and saw that with our |
david-yz-liu
left a comment
There was a problem hiding this comment.
@akarki2005 thanks for doing this. I'm fine with merging in these changes for now, and then please do open a new PR to test these changes.
Proposed Changes
This change seeks to improve the average-case runtime of the CI config in two ways:
Type of Change
(Write an
Xor a brief description next to the type or types that best describe your changes.)Checklist
(Complete each of the following items for your pull request. Indicate that you have completed an item by changing the
[ ]into a[x]in the raw text, or by clicking on the checkbox in the rendered description on GitHub.)Before opening your pull request:
After opening your pull request: