Merge pull request open-telemetry#180 from MSNev/MSNev/UpdateRootPackage

fix: limit npm to <10.0.0 on node v14 and v16
opentelemetrybot · Sep 1, 2023 · c39ce3f · c39ce3f
2 parents 9326e9a + 9e692bb
commit c39ce3f
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/.github/workflows/unit-test.yml b/.github/workflows/unit-test.yml
@@ -23,6 +23,11 @@ jobs:
           node-version: ${{ matrix.node_version }}
 
       - run: npm install -g npm@latest
+        if: ${{ matrix.node_version == '18' || matrix.node_version == '20' }}
+
+      # npm@10.0.0 drops support for Node.js v14 and v16
+      - run: npm install -g npm@"<10.0.0"
+        if: ${{ matrix.node_version == '14' || matrix.node_version == '16' }}
 
       - name: Bootstrap
         run: |
@@ -47,7 +52,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        node_version: [ 16 ]
+        node_version: [ 18 ]
 
     env:
       NPM_CONFIG_UNSAFE_PERM: true
@@ -59,6 +64,7 @@ jobs:
         with:
           node-version: ${{ matrix.node_version }}
 
+      # npm@10.0.0 drops support for Node.js v14 and v16
       - run: npm install -g npm@latest
 
       - name: Bootstrap
@@ -71,5 +77,7 @@ jobs:
           npm run rebuild --verbose
 
       - name: Unit tests
+        env:
+          NODE_OPTIONS: --openssl-legacy-provider
         run: |
           npm run test --verbose