-
Notifications
You must be signed in to change notification settings - Fork 184
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 conditions to test pipeline generation. #1133
Add conditions to test pipeline generation. #1133
Conversation
The following pipelines have been queued for testing: |
Hello @azure-sdk! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
@@ -71,6 +71,7 @@ steps: | |||
--debug | |||
${{parameters.TestsConventionOptions}} | |||
displayName: Create Live Test pipelines for public repository | |||
condition: ne(variables['TestsConventionOptions'],'') |
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.
Should we include succeeded()
to these conditions.
Replicating behavior from our main pipeline generator which will opt out of generating pipelines if there are no options specified.
* Turn warnings as errors for doxygen and add missing docs. * Try to log warnings to a file. * Update script and add comment to internal fields in json. * Update the storage docs. * Change the script to pwsh and write an error message. * Fix remaining warnings. * Only fail if the file is not empty and clean up after and fix typo in a comment. * Fix typo in commment.
Replicating behavior from our main pipeline generator which will opt out of generating pipelines if there are no options specified.