-
Notifications
You must be signed in to change notification settings - Fork 260
Initial Azure Pipelines config #314
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
Codecov Report
@@ Coverage Diff @@
## master #314 +/- ##
=======================================
Coverage 46.48% 46.48%
=======================================
Files 25 25
Lines 3418 3418
=======================================
Hits 1589 1589
Misses 1581 1581
Partials 248 248Continue to review full report at Codecov.
|
jaer-tsun
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.
lgtm, just remove one of the commented lines (restserver test)
|
@jaer-tsun sounds good @sharmasushant If possible, we can switch to Azure Pipelines instead of CircleCI, we just need the app to be installed: I have a sample pr with pipelines enabled on my fork: |
|
@matmerr Remember to fill in the PR template... |
|
/lgtm. But we still need to wait for @sharmasushant's input |
saiyan86
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.
/lgtm
|
Waiting for @sharmasushant's input |
|
yay! |
What this PR does / why we need it:
Replace CircleCI with Azure Pipelines
Special notes for your reviewer:
Azure Pipelines bot also required: https://github.com/marketplace/azure-pipelines
Release note: