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

Node 16.13 (LTS) Support #9698

Closed
Thomasevano opened this issue Nov 18, 2021 · 11 comments
Closed

Node 16.13 (LTS) Support #9698

Thomasevano opened this issue Nov 18, 2021 · 11 comments

Comments

@Thomasevano
Copy link

Hello,

We want to upgrade our app to Node 16.13 but when we start the application, we encounter the following issue: TypeError [ERR_INVALID_ARG_TYPE]: The "options" argument must be of type object. Received an instance of Array

Is Node 16 support in the roadmap ?


Output from ember version --verbose && npm --version && yarn --version:

ember-cli: 3.28.4
node: 16.13.0
v8: 9.4.146.19-node.13
uv: 1.42.0
zlib: 1.2.11
brotli: 1.0.9
ares: 1.17.2
modules: 93
nghttp2: 1.45.1
napi: 8
llhttp: 6.0.4
openssl: 1.1.1l+quic
cldr: 39.0
icu: 69.1
tz: 2021a
unicode: 13.0
ngtcp2: 0.1.0-DEV
nghttp3: 0.1.0-DEV
os: darwin x64
8.1.0
@pichfl
Copy link
Contributor

pichfl commented Nov 18, 2021

See #9694

@Thomasevano
Copy link
Author

Thomasevano commented Nov 18, 2021

See #9694

Yes, i saw it. it remove the warning saying node 16 is not tested, but i don't think it add support

@jherdman
Copy link
Contributor

@Thomasevano I happened to dig into this just last night! As far as I can tell there are certain dependencies using a very ancient version of workerpool. In the case of my app there are two offenders:

  1. ember-cli-flash (via ember-runtime-enumerable-includes-polyfill)
  2. ember-data (we're stuck on v3.24 at this time)

@Thomasevano
Copy link
Author

@Thomasevano I happened to dig into this just last night! As far as I can tell there are certain dependencies using a very ancient version of workerpool. In the case of my app there are two offenders:

  1. ember-cli-flash (via ember-runtime-enumerable-includes-polyfill)
  2. ember-data (we're stuck on v3.24 at this time)

Oh ok, maybe you right, i didn't think of this, cause the error happened in ember-cli-test-loader

@averydev
Copy link

Hi all! 👋 this is something that I'd love to get working -- I have an M1 that doesn't like Node 14 at all 😬

@mansona
Copy link
Member

mansona commented Jan 27, 2022

so this should be fixed since we backported the fix to the "very ancient version of workerpool" https://github.com/josdejong/workerpool/releases/tag/v2.3.4

You might have a lock file that is preventing you from getting that patch release but if you just update that should be fine (with npm the command is just npm update workerpool and that will update the lockfile accordingly if you're on the latest npm)

@runspired
Copy link
Contributor

I notice that as of ember-cli 3.28 we get this warning despite node 16 being the active LTS both then (at time of ember-cli release) and now:

WARNING: Node v16.13.2 is not tested against Ember CLI on your platform. We recommend that you use the most-recent "Active LTS" version of Node.js. See https://git.io/v7S5n for details.

@zavan
Copy link

zavan commented Feb 3, 2022

Thank you @mansona! Upgrading workerpool to 2.3.4 worked for me.

@locks
Copy link
Contributor

locks commented Feb 9, 2022

The original issue has been fixed. Thanks for the discussion!

@locks locks closed this as completed Feb 9, 2022
@runspired
Copy link
Contributor

@locks can you open a new ticket then for adding node16 support to ember-cli?

@locks
Copy link
Contributor

locks commented Feb 12, 2022

@runspired node 16 has been added to the CI matrix and I just tested running an app and didn't have any problems. Can you open the ticket with a reprodduction if you're still having problems?

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

No branches or pull requests

8 participants