From 9e4d1a614a05ec2ca906989f132f9a537bd34042 Mon Sep 17 00:00:00 2001 From: Sima Zhu Date: Mon, 25 Jan 2021 11:12:37 -0800 Subject: [PATCH 1/5] Run the link verification step even previous step failed. --- eng/common/pipelines/templates/steps/verify-links.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/eng/common/pipelines/templates/steps/verify-links.yml b/eng/common/pipelines/templates/steps/verify-links.yml index b23a266ccae..46df511b6dd 100644 --- a/eng/common/pipelines/templates/steps/verify-links.yml +++ b/eng/common/pipelines/templates/steps/verify-links.yml @@ -25,3 +25,4 @@ steps: -branchReplacementName ${{ parameters.BranchReplacementName }} -devOpsLogging: $true -checkLinkGuidance: ${{ parameters.CheckLinkGuidance }} + condition: succeededOrFailed() From 69439c35f6959cf216eace5328241473b9f9f3a5 Mon Sep 17 00:00:00 2001 From: Sima Zhu Date: Mon, 25 Jan 2021 11:18:27 -0800 Subject: [PATCH 2/5] Make the tab right --- eng/common/pipelines/templates/steps/verify-links.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/steps/verify-links.yml b/eng/common/pipelines/templates/steps/verify-links.yml index 46df511b6dd..6350e9e7e7a 100644 --- a/eng/common/pipelines/templates/steps/verify-links.yml +++ b/eng/common/pipelines/templates/steps/verify-links.yml @@ -25,4 +25,4 @@ steps: -branchReplacementName ${{ parameters.BranchReplacementName }} -devOpsLogging: $true -checkLinkGuidance: ${{ parameters.CheckLinkGuidance }} - condition: succeededOrFailed() + condition: succeededOrFailed() From 54a88293629de8b26a24dd25373969523f400c80 Mon Sep 17 00:00:00 2001 From: Sima Zhu Date: Mon, 25 Jan 2021 14:00:19 -0800 Subject: [PATCH 3/5] Address comments --- eng/common/pipelines/templates/steps/verify-links.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/eng/common/pipelines/templates/steps/verify-links.yml b/eng/common/pipelines/templates/steps/verify-links.yml index 6350e9e7e7a..c17f87c417a 100644 --- a/eng/common/pipelines/templates/steps/verify-links.yml +++ b/eng/common/pipelines/templates/steps/verify-links.yml @@ -8,6 +8,7 @@ parameters: Urls: '(Get-ChildItem -Path ./ -Recurse -Include *.md)' BranchReplaceRegex: "^(${env:SYSTEM_PULLREQUEST_SOURCEREPOSITORYURI}.*/(?:blob|tree)/)master(/.*)$" BranchReplacementName: "${env:SYSTEM_PULLREQUEST_SOURCECOMMITID}" + Condition: succeeded() steps: - task: PowerShell@2 @@ -25,4 +26,4 @@ steps: -branchReplacementName ${{ parameters.BranchReplacementName }} -devOpsLogging: $true -checkLinkGuidance: ${{ parameters.CheckLinkGuidance }} - condition: succeededOrFailed() + condition: ${{ parameters.Condition }} From 0cffa3a93f1d303666021d9513011f8966844210 Mon Sep 17 00:00:00 2001 From: Sima Zhu Date: Tue, 26 Jan 2021 11:01:08 -0800 Subject: [PATCH 4/5] Run default condition if no RunOnFailure set. --- eng/common/pipelines/templates/steps/verify-links.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/eng/common/pipelines/templates/steps/verify-links.yml b/eng/common/pipelines/templates/steps/verify-links.yml index c17f87c417a..46f84608f51 100644 --- a/eng/common/pipelines/templates/steps/verify-links.yml +++ b/eng/common/pipelines/templates/steps/verify-links.yml @@ -8,11 +8,12 @@ parameters: Urls: '(Get-ChildItem -Path ./ -Recurse -Include *.md)' BranchReplaceRegex: "^(${env:SYSTEM_PULLREQUEST_SOURCEREPOSITORYURI}.*/(?:blob|tree)/)master(/.*)$" BranchReplacementName: "${env:SYSTEM_PULLREQUEST_SOURCECOMMITID}" - Condition: succeeded() + RunOnFailure: $false steps: - task: PowerShell@2 displayName: Link verification check + condition: or(and(succeededOrFailed(), '${{ parameters.RunOnFailure }}'), and(succeeded(), eq('${{ parameters.RunOnFailure }}', false))) inputs: pwsh: true workingDirectory: '${{ parameters.WorkingDirectory }}/${{ parameters.Directory }}' @@ -26,4 +27,3 @@ steps: -branchReplacementName ${{ parameters.BranchReplacementName }} -devOpsLogging: $true -checkLinkGuidance: ${{ parameters.CheckLinkGuidance }} - condition: ${{ parameters.Condition }} From dd35523799ad0406ad9c972163546727aebc03ad Mon Sep 17 00:00:00 2001 From: Sima Zhu Date: Tue, 26 Jan 2021 15:25:17 -0800 Subject: [PATCH 5/5] Changed back to simple condition --- eng/common/pipelines/templates/steps/verify-links.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/eng/common/pipelines/templates/steps/verify-links.yml b/eng/common/pipelines/templates/steps/verify-links.yml index 46f84608f51..1990ffe07db 100644 --- a/eng/common/pipelines/templates/steps/verify-links.yml +++ b/eng/common/pipelines/templates/steps/verify-links.yml @@ -8,12 +8,12 @@ parameters: Urls: '(Get-ChildItem -Path ./ -Recurse -Include *.md)' BranchReplaceRegex: "^(${env:SYSTEM_PULLREQUEST_SOURCEREPOSITORYURI}.*/(?:blob|tree)/)master(/.*)$" BranchReplacementName: "${env:SYSTEM_PULLREQUEST_SOURCECOMMITID}" - RunOnFailure: $false + Condition: succeeded() # If you want to run on failure for the link checker, set it to `Condition: succeededOrFailed()`. steps: - task: PowerShell@2 displayName: Link verification check - condition: or(and(succeededOrFailed(), '${{ parameters.RunOnFailure }}'), and(succeeded(), eq('${{ parameters.RunOnFailure }}', false))) + condition: ${{ parameters.Condition }} inputs: pwsh: true workingDirectory: '${{ parameters.WorkingDirectory }}/${{ parameters.Directory }}'