-
Notifications
You must be signed in to change notification settings - Fork 40
Optimize CI with already built docker image #617
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
Conversation
d627af1 to
8583d09
Compare
572eba6 to
0a244c2
Compare
70d7e61 to
e1b74ad
Compare
a2e71bc to
384ed2a
Compare
Instead, the coverage step covers the same and will now be mandatory
should be reenabled at some point
Codecov Report❌ Patch coverage is @@ Coverage Diff @@
## main #617 +/- ##
==========================================
- Coverage 77.10% 76.83% -0.28%
==========================================
Files 109 109
Lines 25004 25018 +14
Branches 25004 25018 +14
==========================================
- Hits 19279 19222 -57
- Misses 3708 3772 +64
- Partials 2017 2024 +7
... and 5 files with indirect coverage changes 🚀 New features to boost your workflow:
|
AlexD10S
left a comment
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.
Awesome! thanks for your time and patience on this PR. Looks great; happy to merge.
This PR greatly improves building times in CI by: