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

feat: Drop support for Node.js 12.x #1207

Merged
merged 1 commit into from
Jan 29, 2023

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Jan 28, 2023

BREAKING CHANGE: Minimum supported Node.js version is now 14.x

What:

Drop support for Node.js 12.x

Why:

Reached EOL in April 2022. Tooling is starting to drop Node.js 12.x (see JSDOM) as well so it's time to upgrade to be able to test compat with latest test frameworks (e.g. Jest).

How:

Targetting alpha for now. Want to batch this with #1202
Look for occurence of "12" and update where appropriate. Also introducing Node.js 18.x into test matrix since that is now in Active LTS

Checklist:

  • Documentation added to the
    docs site
  • Tests
  • TypeScript definitions updated
  • Ready to be merged

@eps1lon eps1lon added the BREAKING CHANGE This change will require a major version bump label Jan 28, 2023
@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b092065:

Sandbox Source
react-testing-library-examples Configuration

@codecov
Copy link

codecov bot commented Jan 28, 2023

Codecov Report

Merging #1207 (b092065) into main (0ce0c70) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main     #1207   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           24        24           
  Lines          998       998           
  Branches       327       327           
=========================================
  Hits           998       998           
Flag Coverage Δ
node-12 ?
node-14 100.00% <ø> (ø)
node-16 100.00% <ø> (ø)
node-18 100.00% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@eps1lon eps1lon marked this pull request as ready for review January 28, 2023 10:08
@eps1lon eps1lon changed the base branch from main to alpha January 28, 2023 10:08
@eps1lon eps1lon mentioned this pull request Jan 28, 2023
4 tasks
Copy link
Member

@timdeschryver timdeschryver left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Do you think we should also add v19 to the test matrix?

@eps1lon
Copy link
Member Author

eps1lon commented Jan 29, 2023

Do you think we should also add v19 to the test matrix?

Sure

@eps1lon eps1lon changed the title Drop support for Node.js 12.x feat: Drop support for Node.js 12.x Jan 29, 2023
@eps1lon eps1lon merged commit c37ea2e into testing-library:alpha Jan 29, 2023
@eps1lon eps1lon deleted the feat/drop-node-12 branch January 29, 2023 09:30
@eps1lon eps1lon mentioned this pull request Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BREAKING CHANGE This change will require a major version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants