Skip to content

Commit ff985ef

Browse files
authored
Re-add platform-specific packages for CI (#960)
some packages need to be re-installed on the target machine so that they get the correct implementation. In package-lock.json, we might have arm-version of @nomicfoundation/solidity-analyzer, so that gets installed in CI machine too. However, that machine needs the linux-x64-gnu version of the package. This is fixed by running npm install on those specific packages. Specific error in CI, see e.g. https://github.com/streamr-dev/network-contracts/actions/runs/13110417595/job/36585239309 `HardhatError: HH18: You installed Hardhat with a corrupted lockfile due to the NPM bug #4828.` NomicFoundation/hardhat#4937 suggests reason is "misidentifying the platform in @nomicfoundation/solidity-analyzer" After adding that package, another error appears: `Error: Cannot find module 'solidity-comments-linux-x64-gnu'` After adding that package as well, the run succeeds.
1 parent cac8a12 commit ff985ef

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

.github/workflows/ci.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@ jobs:
3131
- name: Install dependencies
3232
if: steps.cache-node-modules.outputs.cache-hit != 'true'
3333
run: npm ci --no-audit
34+
- name: (Re-)install packages that have platform-specific implementations
35+
run: npm i @nomicfoundation/solidity-analyzer solidity-comments
3436
- name: Build
3537
run: npm run build
3638
- name: Lint

0 commit comments

Comments
 (0)