Merge pull request #6 from curvefi/chore/node

fix: publish permissions
curvefi · Feb 5, 2025 · 0323212 · 0323212
2 parents 1842342 + caad3e3
commit 0323212
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -17,13 +17,14 @@ jobs:
         node-version: [ 20.x, 22.x ]
 
     steps:
+    - name: Checkout repository
+      uses: actions/checkout@v4
+
     - name: Use Node ${{ matrix.node-version }}
       uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
-
-    - name: Checkout repository
-      uses: actions/checkout@v4
+        cache: npm
 
     - name: Clean install
       run: npm ci

diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -38,12 +38,16 @@ jobs:
 
   publish:
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      id-token: write
     steps:
       - uses: actions/checkout@v4
-      - name: Setup .npmrc file to publish to npm
+      - name: Setup node
         uses: actions/setup-node@v4
         with:
-          node-version: 22
+          node-version: '22'
+          cache: npm
       - name: Install modules
         run: npm ci
       - name: Build