Skip to content

Meta: Fix build action#595

Merged
gibson042 merged 1 commit intotc39:masterfrom
gibson042:2021-07-fix-gh-actions
Jul 23, 2021
Merged

Meta: Fix build action#595
gibson042 merged 1 commit intotc39:masterfrom
gibson042:2021-07-fix-gh-actions

Conversation

@gibson042
Copy link
Member

No description provided.

@gibson042 gibson042 merged commit 946aa33 into tc39:master Jul 23, 2021
@ljharb
Copy link
Member

ljharb commented Jul 23, 2021

How does this fix it exactly?

steps:
- name: Checkout 🛎️
uses: actions/checkout@v2
- uses: ljharb/actions/node/run@main
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can switch to node/install and it’d work fine, i suspect - GitHub actions made some breaking changes awhile back

@gibson042
Copy link
Member Author

How does this fix it exactly?

By replacing the old action that started failing, e.g. https://github.com/tc39/ecma402/runs/3139895849

******> nvm install-latest-npm
Attempting to upgrade to the latest working version of npm...
* Installing latest `npm`; if this does not work on your node version, please report a bug!
/usr/local/bin/npm -> /usr/local/lib/node_modules/npm/bin/npm-cli.js
/usr/local/bin/npx -> /usr/local/lib/node_modules/npm/bin/npx-cli.js
npm ERR! code EACCES
npm ERR! syscall mkdir
npm ERR! path /usr/local/share/man/man5
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, mkdir '/usr/local/share/man/man5'

@ljharb
Copy link
Member

ljharb commented Jul 23, 2021

right - node/run → node/install, and moving the command to a separate step, works fine. setup-node isn’t properly maintained.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants