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

What versions of node do we want to support? #217

Closed
trevoro opened this issue Feb 27, 2022 · 2 comments · Fixed by #219
Closed

What versions of node do we want to support? #217

trevoro opened this issue Feb 27, 2022 · 2 comments · Fixed by #219

Comments

@trevoro
Copy link

trevoro commented Feb 27, 2022

Hey,

Lots of the issues in this package seem to be related to the version of node that people are running. The test suite uses node v12 which lots of people have moved past at this point. If we can have a list of LTS releases that we want to support, then we can write the test suite in a way that supports all of them, then we can sort the issues out and bucket them. If anyone runs into a problem, we can just tag it with the version of node they're using and add the test, and that way we can have the library up to date for the known versions.

Here's an example of a test suite we could adopt for the versions that we want; essentially just loop through the different node versions.

WDYT?

@kettanaito
Copy link
Member

Hey, @trevoro. Thanks for raising this.

It seems that a lot of recent issues are related to Node.js versions, just as you say.

I also agree that v12 is archaic and we must migrate from it. I suggest supporting LTS (16.14.0) as the base version. This means migrating all our packages to use that Node.js version, adjusting implementation, etc. I also have a suspicion that it was v16 that introduced at least some breaking changes, as I was able to reproduce a few issues on v16 and v17 with identical errors.

I think that promising support of multiple Node.js versions would be quite a stretch, and suggest that we focus on LTS instead. This project doesn't have the manpower to support it as-is, bringing more Node.js versions support would only make things worse.

If you've got a chance, I'd much appreciate your help with this! We can start by bumping the Node.js version in this repo and opening a pull request with just that. Let's see where the tests fail (if they do!).

@github-actions
Copy link

github-actions bot commented Mar 2, 2022

🎉 This issue has been resolved in version 0.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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 a pull request may close this issue.

2 participants