search

nodejs / llnode

An lldb plugin for Node.js and V8, which enables inspection of JavaScript states for insights into Node.js processes and their core dumps.
Other
1.15k stars 99 forks source link

Fix CI problems #387

Closed oyyd closed 2 years ago

oyyd commented 3 years ago

Currently, tests always fail because:

  1. The action mmarchini/setup-node@mirror uses a deprecated add-path command

Rebasing the upstream code should fix this, like: https://github.com/mmarchini/setup-node/pull/1

  1. Install liblldb-3.9-dev on ubuntu-latest, that is ubuntu-20.04 now, would fail:

image

Don't know why but indicating ubuntu-18.04 should fix this.

No9 commented 2 years ago

Fixed in https://github.com/nodejs/llnode/pull/398

No9 commented 2 years ago

398 landed in main so closing this.

Thanks for reporting