Skip to content

Update version inclusion logic for Node.js parser#107

Open
HarithaVattikuti wants to merge 1 commit into
mainfrom
LTSNodeChange
Open

Update version inclusion logic for Node.js parser#107
HarithaVattikuti wants to merge 1 commit into
mainfrom
LTSNodeChange

Conversation

@HarithaVattikuti
Copy link
Copy Markdown
Contributor

This PR updates the Node.js parser version inclusion logic to match the newer release schedule - Reference

  • keep excluding versions earlier than v8
  • keep including only even-numbered majors for versions below v27
  • include all versions starting from v27

This preserves existing behavior for older releases while allowing newer Node.js versions to be included correctly.

@HarithaVattikuti HarithaVattikuti requested a review from a team as a code owner June 4, 2026 20:22
Copilot AI review requested due to automatic review settings June 4, 2026 20:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Node.js version inclusion rule in NodeVersionsParser to align with the newer Node.js release schedule: preserve the existing “>= v8” floor, keep only even majors for older releases, and include all majors starting at v27.

Changes:

  • Keep excluding Node.js versions with major < 8.
  • Include only even-numbered majors for Node.js versions with major < 27.
  • Include all Node.js versions with major >= 27.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread get-new-tool-versions/parsers/node-parser.psm1
Comment thread get-new-tool-versions/parsers/node-parser.psm1
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.

3 participants