Skip to content

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

Draft
cdcabrera wants to merge 1 commit into
patternfly:mainfrom
cdcabrera:20260427-build
Draft

build(node.js)!: pf-3843 remove 20 support#169
cdcabrera wants to merge 1 commit into
patternfly: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

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
@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

@cdcabrera
Copy link
Copy Markdown
Member Author

cdcabrera commented May 1, 2026

This moves in AFTER a planned 1.1.0 or 1.0.2 release.

@cdcabrera cdcabrera force-pushed the 20260427-build branch 2 times, most recently from 72cdf2d to 67676e9 Compare May 4, 2026 19:51
@cdcabrera
Copy link
Copy Markdown
Member Author

And a new requirement. #194 and potentially #193 need to roll in first since they affect the yml configs

* 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
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