Skip to content

build(node.js)!: pf-3843 remove 20 support#169

Draft
cdcabrera wants to merge 2 commits intopatternfly:mainfrom
cdcabrera:20260427-build
Draft

build(node.js)!: pf-3843 remove 20 support#169
cdcabrera wants to merge 2 commits intopatternfly:mainfrom
cdcabrera:20260427-build

Conversation

@cdcabrera
Copy link
Copy Markdown
Member

@cdcabrera cdcabrera commented Apr 27, 2026

What is it?

  • build(node.js)!: pf-3843 remove 20 support
    • audit, publishing baselines moved to node.js 24
    • build, engine, moved to node.js 22 minimum
    • docs, references moved to node.js 22
    • tests, e2e, cleaned node.js 20 checks

Notes

  • this is considered a breaking change and will push the pf-mcp to a version 2 major
  • scheduled for merge AFTER chore(release): 1.0.1 #168
  • 26 support is scheduled for pf-mcp direct support later this year, but it's non-blocking if someone wants to use it. review our schedule
  • removed the version matrix from publishing workflow. the matrix causes issues in newer release processes.

Unit test check

  1. update the NPM packages with $ npm install
  2. $ npm test
  3. confirm tests come back clean
    • verify that 20 is removed from integration checks leaving only 22 and 24.

Updates issue/story

pf-3843
closes #142

@cdcabrera cdcabrera added the hold review needed label Apr 27, 2026
* audit, publishing baselines moved to node.js 24
* build, engine, moved to node.js 22 minimum
* docs, references moved to node.js 22
* tests, e2e, cleaned node.js 20 checks
@cdcabrera cdcabrera added the breaking a breaking change label Apr 30, 2026
@cdcabrera
Copy link
Copy Markdown
Member Author

Need to make sure we update any refs added by #173

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

Labels

breaking a breaking change hold review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update min Node.js engine requirements

1 participant