Skip to content

chore!: upgrade to Node 18 - #8824

Merged
aabmass merged 1 commit into
googleapis:mainfrom
aabmass:push-xprolqxuswro
Jul 8, 2026
Merged

chore!: upgrade to Node 18#8824
aabmass merged 1 commit into
googleapis:mainfrom
aabmass:push-xprolqxuswro

Conversation

@aabmass

@aabmass aabmass commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

BREAKING CHANGE: Drop support for Node.js versions < 18.

Should have done this in #8822, since pprof has now dropped support for older node versions.

@aabmass
aabmass requested a review from a team as a code owner July 7, 2026 15:56
@aabmass
aabmass requested a review from psx95 July 7, 2026 15:58
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

@aabmass

aabmass commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

/gemini review

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Node.js environment from version 14 to 18 across multiple Kokoro configuration files, test scripts, and package.json engine requirements. There are no review comments, and I have no feedback to provide.

@aabmass

aabmass commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

@danieljbruce after your PR #8811, are the kokoro changes needed? Doesn't seem like these kokoro workflows are run but doesn't hurt to make these changes for consistency.

@aabmass
aabmass force-pushed the push-xprolqxuswro branch 2 times, most recently from aff5859 to fef8d8f Compare July 7, 2026 17:30
@aabmass

aabmass commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

I realized now that the tests were disabled in 9a260ce so I want to re-enable them before release. Will push it to a new PR.

@aabmass
aabmass force-pushed the push-xprolqxuswro branch from fef8d8f to 2b48121 Compare July 8, 2026 15:20
BREAKING CHANGE: Drop support for Node.js versions < 18.
@aabmass
aabmass force-pushed the push-xprolqxuswro branch from 2b48121 to e3c7a35 Compare July 8, 2026 16:48
@aabmass
aabmass merged commit bd61260 into googleapis:main Jul 8, 2026
21 checks passed
@aabmass
aabmass deleted the push-xprolqxuswro branch July 8, 2026 17:00
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.

2 participants