Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions eng/pipelines/templates/jobs/archetype-sdk-client.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,11 @@ jobs:
displayName: "Install dependencies"

- script: |
node common/scripts/install-run-rush.js build --verbose $(GeneratedPackageTargetsTo) $(GeneratedPackageTargetsFrom)
node common/scripts/install-run-rush.js build --verbose $(GeneratedPackageTargets)
displayName: "Build libraries"

- script: |
node common/scripts/install-run-rush.js pack --verbose $(GeneratedPackageTargetsTo) $(GeneratedPackageTargetsFrom)
node common/scripts/install-run-rush.js pack --verbose $(GeneratedPackageTargets)
displayName: "Pack libraries"


Expand Down Expand Up @@ -75,7 +75,7 @@ jobs:
displayName: "Install dependencies"

- script: |
node common/scripts/install-run-rush.js lint $(GeneratedPackageTargetsFrom)
node common/scripts/install-run-rush.js lint $(GeneratedPackageTargets)
displayName: "Lint libraries"

- task: CopyFiles@2
Expand All @@ -93,7 +93,7 @@ jobs:
path: $(Build.ArtifactStagingDirectory)

- script: |
node common/scripts/install-run-rush.js audit $(GeneratedPackageTargetsTo) $(GeneratedPackageTargetsFrom)
node common/scripts/install-run-rush.js audit $(GeneratedPackageTargets)
condition: and(succeeded(), eq(variables['RunNpmAudit'], 'true'))
displayName: "Audit libraries"

Expand Down Expand Up @@ -153,13 +153,13 @@ jobs:
displayName: "Install dependencies"

- script: |
node common/scripts/install-run-rush.js build --verbose $(GeneratedPackageTargetsTo) $(GeneratedPackageTargetsFrom)
node common/scripts/install-run-rush.js build --verbose $(GeneratedPackageTargets)
displayName: "Build libraries"
- script: |
node common/scripts/install-run-rush.js build:test --verbose $(GeneratedPackageTargetsFrom)
node common/scripts/install-run-rush.js build:test --verbose $(GeneratedPackageTargets)
displayName: "Build test assets"
- script: |
node common/scripts/install-run-rush.js unit-test --verbose $(GeneratedPackageTargetsFrom)
node common/scripts/install-run-rush.js unit-test --verbose $(GeneratedPackageTargets)
displayName: "Test libraries"

- task: PublishTestResults@2
Expand Down
20 changes: 5 additions & 15 deletions eng/tools/select-packages/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,7 @@ glob(filter, (err, files) => {
process.exit(1);
}

let packageTargetsTo = "";
let packageTargetsFrom = "";
let packageTargets = "";

if (files) {
log(`Found ${files.length} packages under service directory.`);
Expand All @@ -48,8 +47,7 @@ glob(filter, (err, files) => {
packageContents["sdk-type"]
}".`
);
packageTargetsTo += `--to "${packageContents.name}" `;
packageTargetsFrom += `--from "${packageContents.name}" `;
packageTargets += `--to "${packageContents.name}" `;
} else {
log(
`Package "${
Expand All @@ -60,24 +58,16 @@ glob(filter, (err, files) => {
}

log(
`Finished processing packages. Emitting variable using: ${packageTargetsTo} and ${packageTargetsFrom}`
`Finished processing packages. Emitting variable using: ${packageTargets}`
);

// Can't use regular logging here because the pattern for Azure Pipelines requires ##vso to be the first chars.
console.log(
`##vso[task.setvariable variable=GeneratedPackageTargetsTo]${packageTargetsTo}`
`##vso[task.setvariable variable=GeneratedPackageTargets]${packageTargets}`
);

log(
`Emitted variable "GeneratedPackageTargetsTo" with content: ${packageTargetsTo}`
);

console.log(
`##vso[task.setvariable variable=GeneratedPackageTargetsFrom]${packageTargetsFrom}`
);

log(
`Emitted variable "GeneratedPackageTargetsFrom" with content: ${packageTargetsFrom}`
`Emitted variable "GeneratedPackageTargets" with content: ${packageTargets}`
);

} else {
Expand Down