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

Update node engine requirement #824

Closed
wants to merge 2 commits into from
Closed

Update node engine requirement #824

wants to merge 2 commits into from

Conversation

xg-wang
Copy link
Member

@xg-wang xg-wang commented Apr 7, 2021

https://nodejs.org/en/about/releases/

Release Status Codename Initial Release Active LTS Start Maintenance LTS Start End-of-life
v10 Maintenance LTS Dubnium 2018-04-24 2018-10-30 2020-05-19 2021-04-30
v12 Maintenance LTS Erbium 2019-04-23 2019-10-21 2020-11-30 2022-04-30
v14 Active LTS Fermium 2020-04-21 2020-10-27 2021-10-19 2023-04-30
v15 Current   2020-10-20   2021-04-01 2021-06-01
v16 Pending   2021-04-20 2021-10-26 2022-10-18 2024-04-30

@xg-wang xg-wang added this to the ember-cli-fastboot v3 milestone Apr 7, 2021
@mansona
Copy link
Member

mansona commented May 31, 2021

So i did a little bit of investigating and it seems like there might be somehting wrong with Babel and the promise polyfil 🤔

Screenshot 2021-05-31 at 17 32 14

That is an array that is being passed into validateObject (which fails if you don't pass in an object)

Interestingly I had a different error that I needed to workaround before I even go that far and it was caused by a file that has actually been removed from ember-cli in v3.17, so I wonder if the root of our issue is that we are in a very outdated ember-cli version 🤔

@mansona
Copy link
Member

mansona commented Jul 23, 2021

this was included in #834 👍

@mansona mansona closed this Jul 23, 2021
@xg-wang xg-wang deleted the nodejs branch July 23, 2021 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants