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 to use node version >= 8 #308

Merged
merged 2 commits into from
Jul 18, 2019

Conversation

choheekim
Copy link
Collaborator

ember-exam CI has been failing due to dependency issue with ember-code-snippet. We are using ember-cli-addon-docs has dependency of ember-code-snippet@^2.4.0 in ember-exam and they released new version (2.4.1) with the changes that make the engine incompatible with node version 6.

@scalvert
Copy link
Contributor

👍

This'll need to be a major as it's a breaking change, I believe.

@stefanpenner
Copy link
Contributor

This'll need to be a major as it's a breaking change, I believe.

Yes

@choheekim choheekim force-pushed the node-version-bump branch 3 times, most recently from ca79ae8 to db49a7d Compare June 25, 2019 22:31
…e-snippet. We are using ember-cli-addon-docs has dependency of ember-code-snippet@^2.4.0 in ember-exam and they released new version (2.4.1) with the changes that make the engine incompatible with node version 6.
@choheekim choheekim force-pushed the node-version-bump branch 2 times, most recently from 700a90a to eaf25fe Compare June 26, 2019 00:08
@choheekim
Copy link
Collaborator Author

A pr in ember-cli-addon-docs to bump ember-codes-snippet version should be merged first to resolve current CI failure issue
ember-learn/ember-cli-addon-docs#389

@choheekim choheekim force-pushed the node-version-bump branch from d545172 to eaf25fe Compare July 15, 2019 18:40
package.json Outdated Show resolved Hide resolved
@choheekim choheekim force-pushed the node-version-bump branch from 8e57820 to d2abe64 Compare July 15, 2019 21:57
@step2yeung
Copy link
Collaborator

Just want to verify, we went from ^8.10.0 to 8.*. Was there some original reason we needed ^8.10.0?

@stefanpenner
Copy link
Contributor

:shipit: (assuming we do a major version bump due to node engine change)

@choheekim
Copy link
Collaborator Author

@step2yeung we used ^8.10.0so that we don't support node version 9 and 11 which are not supported. Changing to8.` can cover all 8. version and as well as not supporting version 9 and 11.

@step2yeung step2yeung merged commit 736d121 into ember-cli:master Jul 18, 2019
@step2yeung
Copy link
Collaborator

Merged! Next will release a major version

@choheekim choheekim deleted the node-version-bump branch September 30, 2019 20:13
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

Successfully merging this pull request may close these issues.

4 participants