Skip to content

Commit bb9e22c

Browse files
Lightning00BladeAlinaVarkki
authored andcommitted
chore: fix manual workflow (#312)
The first job was skipped so the second one that was marked as `needs: npm-publish` was getting skipped as well I think this should fix that.
1 parent 85829d9 commit bb9e22c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/publish-to-npm-on-tag.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@ on:
77
workflow_dispatch:
88
inputs:
99
npm-publish:
10-
description: 'Whether or not to try to publish to NPM'
10+
description: 'Try to publish to NPM'
1111
default: false
1212
type: boolean
1313
mcp-publish:
14-
description: 'Whether or not to try to publish to MCP registry'
14+
description: 'Try to publish to MCP registry'
1515
default: true
1616
type: boolean
1717

@@ -22,7 +22,7 @@ permissions:
2222
jobs:
2323
publish-to-npm:
2424
runs-on: ubuntu-latest
25-
if: ${{ github.event_name != 'workflow_dispatch' || inputs.npm-publish }}
25+
if: ${{ (github.event_name != 'workflow_dispatch') || (inputs.npm-publish && always()) }}
2626
steps:
2727
- name: Check out repository
2828
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
@@ -52,8 +52,8 @@ jobs:
5252
5353
publish-to-mcp-registry:
5454
runs-on: ubuntu-latest
55-
if: ${{ github.event_name != 'workflow_dispatch' || inputs.mcp-publish }}
5655
needs: publish-to-npm
56+
if: ${{ (github.event_name != 'workflow_dispatch' && needs.publish-to-npm.result == 'success') || (inputs.mcp-publish && always()) }}
5757
steps:
5858
- name: Check out repository
5959
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0

0 commit comments

Comments
 (0)