-
Notifications
You must be signed in to change notification settings - Fork 24
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
add github action to update the ci image automaticlly #307
Conversation
Signed-off-by: Xiangyu Wang <[email protected]>
03b5c20
to
1c86e12
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅ @@ Coverage Diff @@
## main #307 +/- ##
=======================================
Coverage 49.80% 49.80%
=======================================
Files 331 331
Lines 32705 32705
Branches 4129 4129
=======================================
Hits 16289 16289
Misses 15555 15555
Partials 861 861 |
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
on: | ||
push: | ||
branches: [ "main" ] | ||
|
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.
triggered when merge into main everytime?
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.
yes, for test docker build and push, the trigger condition will be modified again if I am sure it works well
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
resolves: #300