Skip to content

Commit fe78e57

Browse files
ryanbas21claude
andcommitted
fix: gate snapshot and release jobs to their intended triggers
snapshot runs only on workflow_dispatch, release runs only on push to main. Previously both jobs ran on every trigger. Also reverts inputs.extension condition since snapshot is now workflow_dispatch-only. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 7ba9c3d commit fe78e57

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/release.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ concurrency: ${{ github.workflow }}-${{ github.ref }}
1313

1414
jobs:
1515
snapshot:
16+
if: github.event_name == 'workflow_dispatch'
1617
runs-on: ubuntu-latest
1718
permissions:
1819
contents: read
@@ -44,12 +45,12 @@ jobs:
4445
NPM_CONFIG_PROVENANCE: true
4546

4647
- name: Zip extension
47-
if: inputs.extension != false
48+
if: inputs.extension
4849
working-directory: packages/devtools-extension
4950
run: cd dist && zip -r ../extension.zip .
5051

5152
- name: Publish extension to testers
52-
if: inputs.extension != false
53+
if: inputs.extension
5354
uses: mnao305/chrome-extension-upload@v5.0.0
5455
with:
5556
file-path: packages/devtools-extension/extension.zip
@@ -60,6 +61,7 @@ jobs:
6061
publish: true
6162
publish-target: trustedTesters
6263
release:
64+
if: github.event_name == 'push'
6365
runs-on: ubuntu-latest
6466
permissions:
6567
contents: write

0 commit comments

Comments
 (0)