-
Notifications
You must be signed in to change notification settings - Fork 355
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 actions to delfin #552
Conversation
Codecov Report
@@ Coverage Diff @@
## master #552 +/- ##
==========================================
- Coverage 71.16% 71.08% -0.08%
==========================================
Files 138 138
Lines 10524 10629 +105
Branches 1172 1191 +19
==========================================
+ Hits 7489 7556 +67
- Misses 2661 2692 +31
- Partials 374 381 +7
|
1183702
to
9f9b92e
Compare
77fd369
to
feb78f0
Compare
2f97d52
to
98f6f4d
Compare
@@ -1,16 +0,0 @@ | |||
language: python |
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.
let us remove travis.yml after the organisation settings are updated. Otherwise it may impact any PRS coming in between
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.
Already removed
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
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
Actions run is success on local branch
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
@skdwriting please merge this |
98f6f4d
to
699fd79
Compare
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
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
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
CI migration in progress to github actions, local ci ok. hence merging. |
What this PR does / why we need it:
Add Github actions to delfin
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #554Special notes for your reviewer:
Github actions running of the fork: https://github.com/joseph-v/delfin/actions/runs/806330240
Release note: