Skip to content

Commit 8bb1c15

Browse files
authored
Fix code-quality-reports pipeline (Azure#44212)
Fix code-quality-reports pipeline
1 parent ec371ed commit 8bb1c15

File tree

3 files changed

+11
-2
lines changed

3 files changed

+11
-2
lines changed

eng/code-quality-reports/ci.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ trigger:
1111
- .vscode/eclipse-format-azure-sdk-for-java.xml
1212
- eng/code-quality-reports/
1313
- eng/pipelines/code-quality-reports.yml
14+
- eng/pipelines/scripts/Get-Linting-Commands.ps1
15+
- eng/pipelines/scripts/Get-Linting-Reports.ps1
1416

1517
pr:
1618
branches:
@@ -24,6 +26,8 @@ pr:
2426
- .vscode/eclipse-format-azure-sdk-for-java.xml
2527
- eng/code-quality-reports/
2628
- eng/pipelines/code-quality-reports.yml
29+
- eng/pipelines/scripts/Get-Linting-Commands.ps1
30+
- eng/pipelines/scripts/Get-Linting-Reports.ps1
2731

2832
extends:
2933
template: /eng/pipelines/code-quality-reports.yml

eng/pipelines/scripts/Get-Linting-Commands.ps1

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,12 @@ if ($BuildReason -ne "PullRequest") {
5656
}
5757

5858
$diffFiles = (git diff $TargetBranch $SourceBranch --name-only --relative)
59-
if ($diffFiles -contains 'eng/code-quality-reports/ci.yml') {
59+
# Run all linting steps if any of the shared CI configuration files changed.
60+
$runAll = $diffFiles -contains 'eng/code-quality-reports/ci.yml' `
61+
-or $diffFiles -contains 'eng/pipelines/code-quality-reports.yml' `
62+
-or $diffFiles -contains 'eng/pipelines/scripts/Get-Linting-Commands.ps1' `
63+
-or $diffFiles -contains 'eng/pipelines/scripts/Get-Linting-Reports.ps1'
64+
if ($runAll) {
6065
Write-Host "PR changed the CI configuration, running all linting steps."
6166
Write-Host "##vso[task.setvariable variable=${LintingPipelineVariable};]-Dcheckstyle.failOnViolation=false -Dcheckstyle.failsOnError=false -Dspotbugs.failOnError=false -Drevapi.failBuildOnProblemsFound=false"
6267
Write-Host "##vso[task.setvariable variable=RunLinting;]true"

sdk/parents/clientcore-parent/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -958,7 +958,7 @@
958958
</configurationFile>
959959
</analysisConfigurationFiles>
960960
<versionFormat>^\d+\.\d+\.\d+$</versionFormat>
961-
<checkDependencies>true</checkDependencies>
961+
<checkDependencies>false</checkDependencies>
962962
<failBuildOnProblemsFound>${revapi.failBuildOnProblemsFound}</failBuildOnProblemsFound>
963963
<!-- The following configures reporting to file in addition to CLI reporting. -->
964964
<analysisConfiguration>

0 commit comments

Comments
 (0)