-
Notifications
You must be signed in to change notification settings - Fork 204
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
Pipeline work #1053
Pipeline work #1053
Conversation
…perator into azdo-pipelines
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.
Overall changes look good to me
@dkisselev Denis - addressed your comments, need a re-review |
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.
Looks good!
Only note is that the tests in this pipeline fail in Jarrod's azdo because the APIM namespace doesn't exist there - the issue will re-present itself as soon as we move to switch over so it will be a quick fix to update the tests then.
Closes #612
Closes #937
Closes #949
Closes #957
Closes #938
What this PR does / why we need it:
Builds on top of Jarrod's and Denis's pipeline work. This also adds/modifies the following:
Changed back the runs to not tee to a file for report as this removes our ability to track test runs in progress. Kept the coverage report intact.
Special notes for your reviewer:
How does this PR make you feel:
If applicable: