Skip to content

Conversation

@jonkoops
Copy link
Collaborator

BREAKING CHANGE: drops support for all legacy Node.js versions (below version 18)

What kind of change does this PR introduce?

  • Bug Fix
  • Feature
  • Refactoring
  • Style
  • Build
  • Chore
  • Documentation
  • CI

Did you add tests for your changes?

  • Yes, my code is well tested
  • Not relevant

If relevant, did you update the documentation?

  • Yes, I've updated the documentation
  • Not relevant

Summary
Drops support for versions of Node.js that no longer receive active support. This simplifies the CI and makes it easier to do future maintenance.

Does this PR introduce a breaking change?
Yes, this drops support for any version of Node.js older than 18. Realistically though this library is likely not often used in a Node.js environment, so this should not affect many users.

Other information
None.

@coveralls
Copy link

coveralls commented Oct 13, 2024

Pull Request Test Coverage Report for Build 11526169352

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 11523128423: 0.0%
Covered Lines: 89
Relevant Lines: 89

💛 - Coveralls

@jonkoops jonkoops marked this pull request as draft October 21, 2024 10:06
BREAKING CHANGE: drops support for all legacy Node.js versions (below version 18)

Signed-off-by: Jon Koops <[email protected]>
@jonkoops jonkoops marked this pull request as ready for review October 25, 2024 22:19
@jonkoops jonkoops changed the base branch from master to beta October 25, 2024 22:19
@jonkoops
Copy link
Collaborator Author

Marking this one as ready to review. I've rebased the PR and fixed the conflicts. This one is now targeting the beta branch, so if all goes well the CI will release a new version as a pre-release.

@rolandjitsu rolandjitsu merged commit 859310e into react-dropzone:beta Oct 26, 2024
4 checks passed
@github-actions
Copy link

🎉 This PR is included in version 2.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jonkoops jonkoops deleted the drop-old-node branch October 27, 2024 13:06
@jonkoops
Copy link
Collaborator Author

Nice, looks like we successfully published that beta version!

@rolandjitsu
Copy link
Collaborator

Nice, looks like we successfully published that beta version!

Indeed it has. Maintenance branches should also work if/when we decide to patch the last version with commonjs support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants