Skip to content

Commit a77d2da

Browse files
iclantonclaude
andcommitted
Fix YAML parse error by moving condition inside templates
Azure Pipelines doesn't support condition on template calls directly. Move the publishToNpmFeed condition inside the publish.yaml template and pass the parameter from the pipeline files. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 64d95d9 commit a77d2da

3 files changed

Lines changed: 11 additions & 7 deletions

File tree

common/config/azure-pipelines/npm-publish-rush.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,12 +68,12 @@ extends:
6868
parameters:
6969
VersionPolicyName: noRush
7070
BranchName: $(SourceBranch)
71-
condition: eq('${{ parameters.publishToNpmFeed }}', 'true')
71+
PublishToNpmFeed: ${{ parameters.publishToNpmFeed }}
7272

7373
- template: /common/config/azure-pipelines/templates/publish.yaml@self
7474
parameters:
7575
VersionPolicyName: rush
7676
BranchName: $(SourceBranch)
77-
condition: eq('${{ parameters.publishToNpmFeed }}', 'true')
77+
PublishToNpmFeed: ${{ parameters.publishToNpmFeed }}
7878

7979
- template: /common/config/azure-pipelines/templates/post-publish.yaml@self

common/config/azure-pipelines/npm-publish.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,6 @@ extends:
6363
parameters:
6464
VersionPolicyName: noRush
6565
BranchName: $(SourceBranch)
66-
condition: eq('${{ parameters.publishToNpmFeed }}', 'true')
66+
PublishToNpmFeed: ${{ parameters.publishToNpmFeed }}
6767

6868
- template: /common/config/azure-pipelines/templates/post-publish.yaml@self

common/config/azure-pipelines/templates/publish.yaml

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,13 @@ parameters:
44
- name: BranchName
55
type: string
66
default: $(Build.SourceBranchName)
7+
- name: PublishToNpmFeed
8+
type: boolean
9+
default: true
710

811
steps:
9-
- script: 'node common/scripts/install-run-rush.js publish --apply --publish --include-all --target-branch ${{ parameters.BranchName }} --add-commit-details --set-access-level public'
10-
displayName: 'Rush Publish (Policy: ${{ parameters.VersionPolicyName }})'
11-
env:
12-
NPM_AUTH_TOKEN: $(npmToken)
12+
- ${{ if eq(parameters.PublishToNpmFeed, true) }}:
13+
- script: 'node common/scripts/install-run-rush.js publish --apply --publish --include-all --target-branch ${{ parameters.BranchName }} --add-commit-details --set-access-level public'
14+
displayName: 'Rush Publish (Policy: ${{ parameters.VersionPolicyName }})'
15+
env:
16+
NPM_AUTH_TOKEN: $(npmToken)

0 commit comments

Comments
 (0)