-
Notifications
You must be signed in to change notification settings - Fork 10
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
fix: remove all triggers on the api integration test pipeline #382
Conversation
@takla21 I think the commit should use |
Correct. |
184178e
to
3b4a669
Compare
@jeanplevesque @carlh98 I though that |
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.
You could rename the PR to the actual commit :)
3b4a669
to
f546a4a
Compare
@jeanplevesque nvm I misunderstood the new PR checklist. I've made sure to check them all (at least the always applicable) and add a notes in the changelog |
GitHub Issue: #
Proposed Changes
Description
I've made sure that the Api Integration pipeline is not triggered after a commit for any branch.
Impact on version
PR Checklist
Always applicable
No matter your changes, these checks always apply.
README.md
andTemplateConfig.md
if you made changes to templating.AzurePipelines.md
andAPP_README.md
if you made changes to pipelines.Diagnostics.md
if you made changes to diagnostic tools.Architecture.md
and its diagrams if you made architecture decisions or if you introduced new recipes.doc/
folder.Contextual
Based on your changes these checks may not apply.
Other information
Internal Issue (If applicable):