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.

Sign up for GitHub

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

Jump to bottom

fix(js): node executor address already in use #14023

Merged
merged 1 commit into from Dec 27, 2022
Merged

fix(js): node executor address already in use #14023

merged 1 commit into from Dec 27, 2022

Commits on Dec 26, 2022

  1. fix(js): in new map(key, value), key cannot be an array as it is retu… 

    …rning undefined. issue#14017

There was a breaking change done on @nrwl/js package through #13813
PR, this has caused an issue#14017 on the frameworks like express, nest.js and wherever
@nrwl/js:node is used as executor.
Once the fresh installation of express or nest.js is done
through nx, after every file changes, it gives EADDRINUSE error.
The issue is caused by using
\"array\" as key in \"new Map(key, value)\" function. I corrected it by using JSON.stringify function.

BREAKING CHANGE:
NA

ISSUES CLOSED: #14017
    Sahil Purav committed Dec 26, 2022
    Copy the full SHA
    9385ce7 View commit details
    Browse the repository at this point in the history