Skip to content

ci: use matrix and cache to speed up the build#282

Merged
koordinator-bot[bot] merged 1 commit intokoordinator-sh:mainfrom
jasonliu747:cache
Jun 21, 2022
Merged

ci: use matrix and cache to speed up the build#282
koordinator-bot[bot] merged 1 commit intokoordinator-sh:mainfrom
jasonliu747:cache

Conversation

@jasonliu747
Copy link
Member

Signed-off-by: Jason Liu jasonliu747@gmail.com

Ⅰ. Describe what this PR does

save some time from building docker images.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it

Ⅳ. Special notes for reviews

@codecov
Copy link

codecov bot commented Jun 21, 2022

Codecov Report

Merging #282 (2f848fd) into main (c5ee5fa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #282   +/-   ##
=======================================
  Coverage   61.06%   61.06%           
=======================================
  Files         103      103           
  Lines        9599     9599           
=======================================
  Hits         5862     5862           
  Misses       3250     3250           
  Partials      487      487           
Flag Coverage Δ
unittests 61.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5ee5fa...2f848fd. Read the comment docs.

@jasonliu747 jasonliu747 force-pushed the cache branch 8 times, most recently from 501ef24 to 923152c Compare June 21, 2022 10:58
@jasonliu747 jasonliu747 changed the title ci: use cache to speed up the build ci: use matrix and cache to speed up the build Jun 21, 2022
Signed-off-by: Jason Liu <jasonliu747@gmail.com>
@hormes
Copy link
Member

hormes commented Jun 21, 2022

/lgtm
/approve

@koordinator-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hormes

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jasonliu747 jasonliu747 added this to the v0.5 milestone Jun 21, 2022
@koordinator-bot koordinator-bot bot merged commit c4e2272 into koordinator-sh:main Jun 21, 2022
@jasonliu747
Copy link
Member Author

Before optimization:
image
After:
image

@jasonliu747 jasonliu747 deleted the cache branch June 21, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants