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

bump node version to 16 #741

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jimmywarting
Copy link

This update raises the minimum required Node.js version to v16. This change enables us to embrace newer technologies without hesitation and eliminates the need to consider whether it's acceptable to utilize modern solutions or resort to older code styles. This enhancement not only improves the overall development experience but also encourages contributions if they can contribute with newer modern code.

fix #739

@SimenB
Copy link
Member

SimenB commented Feb 22, 2024

This should be fine to do, but I'm not sure if a breaking change just to bump the node version is needed (unless it also allows us to e.g. update some dependencies or some such)

@bcoe
Copy link
Member

bcoe commented Mar 21, 2024

@jimmywarting bother you to update this to Node 18, now that 16 is EOL?

@SimenB given we're jumping so many version #s as a minimum engine #, I think this should be a breaking change (upstream users can choose not to bump their major).

@SimenB
Copy link
Member

SimenB commented Mar 21, 2024

yeah, definitely a breaking change regardless of which versions are dropped. My question was more - what does it give us to drop old versions? Dropping them "just because" doesn't seem valuable

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.

Upgrade Node.js LTS Version and package.json Update
3 participants