search

ros-tooling / action-ros-lint

Github action to run linters on ROS 2 packages
Apache License 2.0
19 stars 7 forks source link

Update test.yml #372

Closed mosfet80 closed 3 months ago

mosfet80 commented 3 months ago

updated libs switch to node20 -->partial fix from node 12 to node 16.

For now I have reduced the patch version from 12 to 16.

christophebedard commented 3 months ago

I don't know why CI is not running.

christophebedard commented 3 months ago

OK, I figured it out. The workflow had been disabled due to inactivity here: https://github.com/ros-tooling/action-ros-lint/actions/workflows/test.yml.

christophebedard commented 3 months ago

I've seen this error before when trying to bump Node.js version:

Error: EACCES: permission denied, open 'dist/index.js'
    at Object.openSync (node:fs:590:3)
    at writeFileSync (node:fs:2202:35)
    at handler (/__w/action-ros-lint/action-ros-lint/node_modules/@zeit/ncc/dist/ncc/cli.js.cache.js:1:48039) {
  errno: -13,
  syscall: 'open',
  code: 'EACCES',
  path: 'dist/index.js'
}

See: https://github.com/ros-tooling/setup-ros/pull/521#issuecomment-1427093726

I think this was the fix: https://github.com/ros-tooling/setup-ros/pull/521/commits/3710723f7aa4b2f87dd7d60527740721368ab48a