Skip to content

Commit 2685668

Browse files
Fixup to new ESRP publish pipeline (#9309)
- package-publish require approval to use the environment which breaks our flow - add condition on the whole stage so it shows as skip if no package needs publish -
1 parent 97bad6b commit 2685668

2 files changed

Lines changed: 3 additions & 9 deletions

File tree

eng/tsp-core/pipelines/jobs/publish-npm.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
displayName: Publish Npm (${{ parameters.tag }})
1515
dependsOn: ${{ parameters.dependsOn }}
1616
condition: ${{ parameters.condition }}
17-
environment: package-publish
17+
environment: none
1818
pool: # hardcoding the pool and image name because deployment jobs do not support variable expansion in pool names
1919
name: azsdk-pool
2020
image: ubuntu-24.04
@@ -25,7 +25,7 @@ jobs:
2525
inputs:
2626
- input: pipelineArtifact
2727
artifactName: ${{ parameters.artifactName }}
28-
itemPattern: "**/*.{tgz}"
28+
itemPattern: "**/*.tgz"
2929
targetPath: $(Pipeline.Workspace)/${{ parameters.artifactName }}/
3030

3131
strategy:

eng/tsp-core/pipelines/publish.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -42,20 +42,14 @@ extends:
4242
- stage: publish_npm
4343
displayName: Publish Npm Packages
4444
dependsOn: build
45+
condition: or(eq('true', dependencies.build.outputs['build.pack_npm_packages_stable.PUBLISH_PKG_ANY']), eq('true', dependencies.build.outputs['build.pack_npm_packages_next.PUBLISH_PKG_ANY']))
4546

4647
pool:
4748
name: $(LINUXPOOL)
4849
image: $(LINUXVMIMAGE)
4950
os: linux
5051

5152
jobs:
52-
- job: debug
53-
variables:
54-
varFromA: $[convertToJson(stageDependencies.build.outputs)]
55-
deps: $[convertToJson(stageDependencies)] # create a variable with the job dependencies
56-
steps:
57-
- pwsh: Write-Host "$(deps)"
58-
5953
- template: /eng/tsp-core/pipelines/jobs/publish-npm.yml
6054
parameters:
6155
artifactName: npm-packages-stable

0 commit comments

Comments
 (0)