Skip to content

Commit ffa6e57

Browse files
authored
Merge pull request #50 from Expensify/andrew-oidc
Update node and permissions to allow publishing via OIDC
2 parents df1e7a2 + a4bedad commit ffa6e57

File tree

3 files changed

+7
-2
lines changed

3 files changed

+7
-2
lines changed

.github/workflows/publish.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,11 @@ on:
44
push:
55
branches: [main]
66

7+
permissions:
8+
# Required for OIDC: https://docs.npmjs.com/trusted-publishers
9+
id-token: write
10+
contents: read
11+
712
# Ensure that only one instance of this workflow executes at a time.
813
# If multiple PRs are merged in quick succession, there will only ever be one publish workflow running and one pending.
914
concurrency: ${{ github.workflow }}

.github/workflows/style_and_syntax_checks.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ jobs:
1616
# v3
1717
uses: actions/setup-node@1a4442cacd436585916779262731d5b162bc6ec7
1818
with:
19-
node-version: 18
19+
node-version-file: '.nvmrc'
2020
cache: npm
2121

2222
- name: Install dependencies

.nvmrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
20.10.0
1+
24.11.0

0 commit comments

Comments
 (0)