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 the setup to node 18 #441

Merged
merged 5 commits into from
Oct 10, 2023
Merged

Update the setup to node 18 #441

merged 5 commits into from
Oct 10, 2023

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Oct 9, 2023

This updates the dev and CI setup to node 18. node 14 (and 16) are both EOL and some tools (Prettier for example) start to depend on newer features.

This only affects the dev setup. The addon doesn't really do any node logic, so it will have little influence on consumers.

This updates the dev and CI setup to node 18. node 14 (and 16) are both EOL and some tools (Prettier for example) start to depend on newer features.

This only affects the dev setup. The addon doesn't really do any node logic, so it will have little influence on consumers.
This removes some dependencies that are no longer needed and forces the ember-source peerDep of storybook/ember to our own version.
- ember-focus-trap v1 has an Ember v4+ peerDependency
- storybook has an Ember 3.28 peerDep
- zestia/autofocus v4 doesn't support ember-source v5 yet as a peerDep
This works around an issue in node 18.18 that makes the embroider-optimized scenario hang.
@Windvis Windvis mentioned this pull request Oct 10, 2023
@Windvis Windvis merged commit 224f40c into master Oct 10, 2023
13 checks passed
@Windvis Windvis deleted the chore/node-18-dev-setup branch October 10, 2023 08:43
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.

1 participant