Skip to content

Commit 3699842

Browse files
authored
chore(ci): fix failure notify job conditional in publish workflow (vectordotdev#17468)
The syntax of the conditional for the publish workflow's failure notification job had a bug where an explicit expression was defined and was paired with an implicit expression. This resulted in the job running when it shouldn't. Removed the explicit expression syntax from the other job conditionals to avoid potentially doing that in the future.
1 parent 78bbfbc commit 3699842

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/publish.yml

+4-4
Original file line numberDiff line numberDiff line change
@@ -548,7 +548,7 @@ jobs:
548548
publish-github:
549549
name: Publish to GitHub
550550
# We only publish to GitHub for versioned releases, not nightlies.
551-
if: ${{ inputs.channel == 'release' }}
551+
if: inputs.channel == 'release'
552552
runs-on: ubuntu-20.04
553553
needs:
554554
- generate-publish-metadata
@@ -618,7 +618,7 @@ jobs:
618618
publish-homebrew:
619619
name: Publish to Homebrew
620620
# We only publish to Homebrew for versioned releases, not nightlies.
621-
if: ${{ inputs.channel == 'release' }}
621+
if: inputs.channel == 'release'
622622
runs-on: ubuntu-20.04
623623
needs:
624624
- generate-publish-metadata
@@ -638,7 +638,7 @@ jobs:
638638
publish-cloudsmith:
639639
name: Publish to Cloudsmith
640640
# We only publish to CloudSmith for versioned releases, not nightlies.
641-
if: ${{ inputs.channel == 'release' }}
641+
if: inputs.channel == 'release'
642642
runs-on: ubuntu-20.04
643643
needs:
644644
- generate-publish-metadata
@@ -751,7 +751,7 @@ jobs:
751751

752752
publish-failure:
753753
name: Send Publish Failure Notification
754-
if: ${{ inputs.channel != 'custom' }} && failure()
754+
if: failure() && inputs.channel != 'custom'
755755
runs-on: ubuntu-20.04
756756
needs:
757757
- generate-publish-metadata

0 commit comments

Comments
 (0)