Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pnpm deploy now fails with MaxListenersExceededWarning #6510

Closed
larsivi opened this issue May 6, 2023 · 1 comment · Fixed by #6511
Closed

pnpm deploy now fails with MaxListenersExceededWarning #6510

larsivi opened this issue May 6, 2023 · 1 comment · Fixed by #6511

Comments

@larsivi
Copy link

larsivi commented May 6, 2023

Last pnpm version that worked:

8.3.1

pnpm version:

8.4.0

Code to reproduce the issue:

pnpm --filter=something deploy deployfolder

Expected behavior:

It completes the command, producing the deployable content

Actual behavior:

It prints a bunch of (one for every workspace dependency I think)
The modules directory at "/home/foo/code/bar/deploy/node_modules" will be removed and reinstalled from scratch. Proceed? (Y/n)

It then prints the messages below, then hangs
(node:277420) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 keypress listeners added to [ReadStream]. Use emitter.setMaxListeners() to increase limit
(Use node --trace-warnings ... to show where the warning was created)
(node:277420) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 end listeners added to [ReadStream]. Use emitter.setMaxListeners() to increase limit
(node:277420) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added to [ReadStream]. Use emitter.setMaxListeners() to increase limit

Additional information:

  • node -v prints:
    v18.14.2

  • Windows, macOS, or Linux?:
    Linux, recent KDE Neon (Ubuntu based)

@zkochan
Copy link
Member

zkochan commented May 7, 2023

Probably we should not display the prompt for the deploy command.

As a workaround, try running it with the --force option: pnpm --force --filter=something deploy deployfolder

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

Successfully merging a pull request may close this issue.

2 participants