Overview

It's failing on multiple jobs - e.g. https://git.drupalcode.org/issue/experience_builder-3458683/-/jobs/2164784

$ npm ci
npm warn deprecated rimraf@3.0.2: Rimraf versions prior to v4 are no longer supported
npm warn deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated glob@7.2.3: Glob versions prior to v9 are no longer supported
npm error Exit handler never called!
npm error This is an error with npm itself. Please report this error at:
npm error   <https://github.com/npm/cli/issues>
npm error A complete log of this run can be found in: /root/.npm/_logs/2024-07-19T03_13_15_531Z-debug-0.log
$ npm run build
> vite-template-redux@0.0.0 build
> npm run type-check && vite build
> vite-template-redux@0.0.0 type-check
> tsc --noEmit --pretty
sh: tsc: not found
Cleaning up project directory and file based variables
00:01
ERROR: Job failed: command terminated with exit code 127
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

jessebaker created an issue. See original summary.

wim leers’s picture

Status: Active » Reviewed & tested by the community

Aha! node:current-alpine resolves to node 22 currently, and not all of our deps work on that new version — is that right?

Obviously this unbreaks the build, so: 🚢

jessebaker’s picture

node:current-alpine resolves to node 22 currently, and not all of our deps work on that new version — is that right?

That is exactly right - I'm running npm ci on node 22 locally and it just hangs so it's hard to figure out what specifically is breaking it. Merging now to unblock everything.

  • jessebaker committed f1cf2f77 on 0.x
    Issue #3462584 by jessebaker: NPM Warning on UI CI pipeline preventing...
wim leers’s picture

Status: Reviewed & tested by the community » Needs review

#4: oh wow, that sounds alarming!

+1 for having merged this MR to unblock everything immediately 👍👏

But reading #4, perhaps we should add a second MR to this issue to narrow ui/package.json's

 "engines": {
    "node": ">=20.0.0"
  },

to only version 20? 🤔

wim leers’s picture

Assigned: jessebaker » bnjmnm
Status: Needs review » Reviewed & tested by the community

Needs @bnjmnm's approval to merge this — Trivial MR of the Month to review for you, @bnjmnm 😄

bnjmnm made their first commit to this issue’s fork.

  • bnjmnm committed 6c696bec on 0.x authored by jessebaker
    Issue #3462584: NPM Warning on UI CI pipeline preventing build 
    
wim leers’s picture

Assigned: bnjmnm » Unassigned
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.