Skip to content

feat(chore): remove Node js 20 support#109

Merged
yeshamavani merged 2 commits into
masterfrom
GH-108
Jun 18, 2026
Merged

feat(chore): remove Node js 20 support#109
yeshamavani merged 2 commits into
masterfrom
GH-108

Conversation

@Sourav-kashyap

@Sourav-kashyap Sourav-kashyap commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

BREAKING CHANGE:
yes

GH-108

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-12 at 5 08 53 PM

Test:

Screenshot 2026-06-12 at 5 13 28 PM

@Sourav-kashyap Sourav-kashyap self-assigned this Jun 12, 2026
@Sourav-kashyap Sourav-kashyap linked an issue Jun 12, 2026 that may be closed by this pull request
@Sourav-kashyap Sourav-kashyap force-pushed the GH-108 branch 2 times, most recently from 2a97c42 to 7bf7f1c Compare June 16, 2026 09:05
BREAKING CHANGE:
yes

GH-108
@sonarqubecloud

Copy link
Copy Markdown

SonarQube reviewer guide

Summary: Drop Node.js 20 support and update the minimum required version to Node.js 22 or 24, along with dependency upgrades and configuration file updates.

Review Focus: The most critical change is the shift from supporting Node.js 20 as the minimum version to requiring Node.js 22 or 24. This is a breaking change for any users running on Node.js 20 or earlier. Additionally, there are extensive dependency updates throughout the package-lock.json file. Pay particular attention to:

  1. Breaking changes in upgraded packages (e.g., helmet 5.1.1 → 8.2.0, express-rate-limit 6.11.2 → 8.5.2)
  2. New dependencies being added (form-data)
  3. Engine requirements in updated packages to ensure they're compatible with Node.js 22+

Start review at: package.json. This is where the Node.js version constraint change is declared in the engines field (changing from >=20 to 22 || 24), which is the primary functional change of this PR. Understanding this requirement will help contextualize all the downstream dependency updates in the lock file.

💬 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

@yeshamavani yeshamavani merged commit 5f460ea into master Jun 18, 2026
6 checks passed
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

2 participants