Skip to content
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

Update conditions.md #13463

Merged
merged 1 commit into from
Aug 11, 2023
Merged

Update conditions.md #13463

merged 1 commit into from
Aug 11, 2023

Conversation

akoppanur
Copy link
Contributor

  1. In line 129, Stage 1 does not have dependsOn: stage2. By default Stage 2 depends on stage1. So, in the below statement, it should be 'stage2' depends on 'stage1'

In this pipeline, stage1 depends on stage2. Job B has a condition set for it.

  1. In Line 146, the 'Stage2' has Job B. So in the below statement, the job in 'stage2' should be job 'B' and not 'A'

If you queue a build on the main branch, and you cancel it while stage1 is running, stage2 won't run, even though it contains a job B whose condition evaluates to true.

In Line 146, the 'Stage2' has Job B. So in the below statement, the job in 'stage2' should be job 'B' and not 'A'

---------------
If you queue a build on the `main` branch, and you cancel it while `stage1` is running, `stage2` *won't* run, even though it contains a job `B` whose condition evaluates to `true`.
---------------
@prmerger-automator
Copy link
Contributor

@akoppanur : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Court72
Copy link
Contributor

Court72 commented Jul 7, 2023

@juliakm

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@juliakm
Copy link
Collaborator

juliakm commented Aug 11, 2023

@akoppanur Thanks for your detailed edits! It's contributors like you that make our documentation better. I appreciate your PR.

#sign-off

@Court72 Court72 merged commit 9749835 into MicrosoftDocs:main Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants