Skip to content

Commit 191d02d

Browse files
authored
add "release" trigger to workflows (#24)
* add "release" trigger to workflows * remove auto-publish on release seems to trigger incorrectly? manual workflow_dispatch -> release is fine for now.
1 parent 3b28c30 commit 191d02d

File tree

2 files changed

+14
-2
lines changed

2 files changed

+14
-2
lines changed

.github/workflows/build-js.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,12 @@ on:
1111
- '**.md'
1212
- 'python/**'
1313
workflow_dispatch:
14+
inputs:
15+
publish:
16+
description: 'Should packages be published to npm?'
17+
required: true
18+
default: false
19+
type: boolean
1420

1521
jobs:
1622
client-build:
@@ -61,7 +67,7 @@ jobs:
6167
client-publish:
6268
name: Publish JavaScript SDK
6369
runs-on: ubuntu-latest
64-
if: github.event_name == 'release' && github.event.action == 'published'
70+
if: ${{ inputs.publish }}
6571
needs: [client-build]
6672
steps:
6773
- uses: actions/checkout@v3

.github/workflows/build-python.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,12 @@ on:
1515
- '**.md'
1616
- 'js/**'
1717
workflow_dispatch:
18+
inputs:
19+
publish:
20+
description: 'Should packages be published to PyPI?'
21+
required: true
22+
default: false
23+
type: boolean
1824

1925
permissions:
2026
contents: read
@@ -120,7 +126,7 @@ jobs:
120126
publish:
121127
name: Publish
122128
runs-on: ubuntu-latest
123-
if: github.event_name == 'release' && github.event.action == 'published'
129+
if: ${{ inputs.publish }}
124130
needs: [linux, macos, windows, sdist]
125131
steps:
126132
- uses: actions/download-artifact@v3

0 commit comments

Comments
 (0)