npm needs to be bumped to 8.8.0, see nodejs/node#42886

Thanks @branchvincent - sorry I thought the PR hadn't gone through due to a duplicate.

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a help wanted or in progress label.

This is unfortunate. Can we please reopen the PR? Was there a CI failure that blocked bumping the formula?

Error: PR requires the CI-long-timeout label but it is not set!
Error: If the longer timeout is not required, remove the "long build" label.
Error: Otherwise, add the "CI-long-timeout" label.
Error: No more than 2 PRs at a time may use "CI-long-timeout".

Can you please add the CI-long-timeout label, @branchvincent?

@derrabus There is only a limited amount of long-timeout slots available, so CI cannot be restarted at the moment.
The failures from the previous run should be addressed first anyway - https://github.com/Homebrew/homebrew-core/actions/runs/2266546607

Thanks, those failures were the piece of information I was looking for. 🙂

To summarize those logs, all runners encountered the same two issues:

• bash-language-server

==> /usr/local/Cellar/bash-language-server/2.1.0/bin/bash-language-server start
  (node:58791) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
  (Use `node --trace-warnings ...` to show where the warning was created)
  node:internal/deps/undici/undici:4816
              throw new TypeError("Failed to parse URL from " + input, { cause: err });
                    ^
  
  TypeError: Failed to parse URL from /usr/local/Cellar/bash-language-server/2.1.0/libexec/lib/node_modules/bash-language-server/node_modules/web-tree-sitter/tree-sitter.wasm

• emscripten

==> node test.js
  (node:79957) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
  (Use `node --trace-warnings ...` to show where the warning was created)
  /private/tmp/emscripten-test-20220504-77343-3id2oy/test.js:146
        throw ex;
        ^
  
  TypeError: Failed to parse URL from /private/tmp/emscripten-test-20220504-77343-3id2oy/test.wasm

Seems to have been tracked upstream

nodejs/node#42701 (comment)

The test for emscripten will pass with the flag --no-experimental-fetch - emscripten-core/emscripten#16913

#101576 update should fix bash-language-server

@branchvincent Do we usually fix the tests in a separate PR? I think this should be green now.

Could we merge this?

Still the same test error on bash-language-server, was supposed to be fixed in version 3.0.1 but seems that it hasn't been.

node 18.2.0 has been released, bumping NPM to 8.9.0: https://github.com/nodejs/node/releases/tag/v18.2.0

maybe let's merge this one and fix the failures and then start 18.2.0 journey.

@chenrui333 has triggered a merge.

only bash-language-server failed on all builds, I will do a merge and follow up to see if that is reproducible.