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

Drops support for Node 10.x, 12.x and adds 18.x #723

Merged
merged 2 commits into from Sep 8, 2022

Conversation

popematt
Copy link
Contributor

@popematt popematt commented Sep 8, 2022

Issue #, if available:

None

Description of changes:

This updates the supported versions of Node.js and bumps the version to 5.0.0-SNAPSHOT (since dropping support for old node is theoretically a backwards incompatible change).

Note that this PR is against the v5.0.0-development branch, not against master.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@popematt
Copy link
Contributor Author

popematt commented Sep 8, 2022

ion-test-driver failure is independent of my changes. Either way, my changes would not affect the outcome of those tests.

matrix:
node-version: [10.x, 12.x, 14.x, 16.x]
node-version: [14.x, 16.x, 18.x]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a quick question here. According to the README.md, should this node-version also include 20?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My mistake. README is wrong—Node 20 hasn't been released yet.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My mistake. README is wrong—Node 20 hasn't been released yet.

Other than that, looks good to me.

@popematt popematt merged commit 9ee772d into amazon-ion:v5.0.0-development Sep 8, 2022
desaikd added a commit that referenced this pull request Apr 24, 2023
* Drops support for Node 10.x, 12.x and adds 18.x (#723)

* move to native `BigInt` from `JSBI` (#721)

* Updates typedoc syntax to address warnings (#733)

* updates test-driver to es2020 (#736)

* Migrate linting from TSLint to ESLint (#737)

* adds changes for resolving ion-test-driver build issue (#748)

* adds changes to allow `dom.Decimal` construction from `Number` and `String` (#746)

* updates uglify version

* adds changes for upconverting JS value for dom collections (#749)

* removes deprecated method `byteValue()` for v5.0.0 release (#750)

* modifies `Struct#elements()` to return all field values (#754)

---------

Co-authored-by: Matthew Pope <81593196+popematt@users.noreply.github.com>
Co-authored-by: Delaney <delaneygillilan@gmail.com>
Co-authored-by: Zack Slayton <zack.slayton@gmail.com>
Co-authored-by: Andrey Lipatkin <400234+Litee@users.noreply.github.com>
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.

None yet

2 participants