Skip to content

Commit cd6591a

Browse files
committed
fix: restore release workflow env context from main
The rebase left a stale release.yml that used secrets context in step if-conditions instead of job-level env context. This would break workflow_dispatch and overwrite the fix from PR #75 on merge. Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
1 parent a89fd12 commit cd6591a

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

.github/workflows/release.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,9 @@ jobs:
4141
timeout-minutes: 15
4242
permissions:
4343
contents: write
44+
env:
45+
VSCE_PAT: ${{ secrets.VSCE_PAT }}
46+
OVSX_PAT: ${{ secrets.OVSX_PAT }}
4447
steps:
4548
- uses: step-security/harden-runner@0634a2670c59f64b4a01f0f96f84700a4088b9f0 # v2.12.0
4649
with:
@@ -60,12 +63,8 @@ jobs:
6063
# one-time browser setup (aex.dev.azure.com, PAT scopes, Open VSX agreement
6164
# + create-namespace + ownership claim), secret names, and troubleshooting.
6265
- name: Publish to VS Code Marketplace
63-
if: ${{ secrets.VSCE_PAT != '' }}
66+
if: env.VSCE_PAT != ''
6467
run: npx @vscode/vsce publish --packagePath patchloom.vsix
65-
env:
66-
VSCE_PAT: ${{ secrets.VSCE_PAT }}
6768
- name: Publish to Open VSX
68-
if: ${{ secrets.OVSX_PAT != '' }}
69+
if: env.OVSX_PAT != ''
6970
run: npx ovsx publish patchloom.vsix
70-
env:
71-
OVSX_PAT: ${{ secrets.OVSX_PAT }}

0 commit comments

Comments
 (0)