Skip to content

refactor(chore): remove Node js 20 support#275

Open
Sourav-kashyap wants to merge 1 commit into
masterfrom
GH-274
Open

refactor(chore): remove Node js 20 support#275
Sourav-kashyap wants to merge 1 commit into
masterfrom
GH-274

Conversation

@Sourav-kashyap

Copy link
Copy Markdown
Contributor

remove Node js 20 support

BREAKING CHANGE:
yes

GH-274

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Checklist:

  • Performed a self-review of my own code
  • npm test passes on your machine

Build:

Screenshot 2026-06-11 at 6 09 16 PM

Test:

Screenshot 2026-06-11 at 6 09 36 PM

remove Node js 20 support

BREAKING CHANGE:
yes

GH-274
@sonarqubecloud

Copy link
Copy Markdown

SonarQube reviewer guide

Summary: Drops support for Node.js 20 and updates dependencies to v10.1.0.

Review Focus: The primary changes are removing Node.js 20 from the test matrix and updating the minimum Node.js engine requirement to 22. Pay attention to whether these dependency updates introduce any breaking changes or compatibility issues with the new minimum Node version. The large number of dependency updates (especially AWS SDK and Smithy packages) suggests potential API changes that may impact the S3 module functionality.

Start review at: package.json. This file contains the critical engine requirement change from Node 20 to Node 22+, and also shows the version bump to 10.1.0. Verify the engine constraint is appropriate for your deployment targets and review the rationale for dropping Node.js 20 support, as this is a breaking change for users still on that version.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Sourav-kashyap Sourav-kashyap linked an issue Jun 11, 2026 that may be closed by this pull request
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.

remove Node.js 20 support

1 participant