Skip to content

Commit 4ecf1fe

Browse files
authored
Add SDKType check for aggregate coverge report (Azure#18430)
* Add SDKType check for aggregate coverge report * Fix condition * Fix syntax
1 parent f463b5a commit 4ecf1fe

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

eng/pipelines/templates/jobs/archetype-sdk-client.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ jobs:
312312

313313
- task: Maven@3
314314
displayName: 'Generate aggregate code coverage report'
315-
condition: and(eq(variables['OsName'], 'Windows'), eq(variables['JavaTestVersion'], '1.11'))
315+
condition: and(eq(variables['OsName'], 'Windows'), eq(variables['JavaTestVersion'], '1.11'), eq('${{ parameters.SDKType }}', 'client'))
316316
inputs:
317317
mavenPomFile: sdk/${{ parameters.ServiceDirectory }}/pom.xml
318318
options: -Pcoverage
@@ -374,7 +374,7 @@ jobs:
374374
# Azure DevOps only seems to respect the last code coverage result published, so only do this for Windows + Java LTS.
375375
# Code coverage reporting is setup only for Track 2 modules.
376376
- task: PublishCodeCoverageResults@1
377-
condition: and(eq(variables['OsName'], 'Windows'), eq(variables['JavaTestVersion'], '1.11'))
377+
condition: and(eq(variables['OsName'], 'Windows'), eq(variables['JavaTestVersion'], '1.11'), eq('${{ parameters.SDKType }}', 'client'))
378378
inputs:
379379
codeCoverageTool: JaCoCo
380380
summaryFileLocation: sdk/${{ parameters.ServiceDirectory }}/target/site/jacoco-aggregate/jacoco.xml

0 commit comments

Comments
 (0)