Skip to content

feat!: Update common to minimum Node version of 22. - #8988

Merged
danieljbruce merged 2 commits into
mainfrom
node-22-common
Jul 29, 2026
Merged

feat!: Update common to minimum Node version of 22.#8988
danieljbruce merged 2 commits into
mainfrom
node-22-common

Conversation

@danieljbruce

Copy link
Copy Markdown
Contributor

Description

Upgrade the Node version from 18 to 22.

Towards #8985

@danieljbruce
danieljbruce requested a review from a team as a code owner July 28, 2026 15:32
@github-actions
github-actions Bot requested a review from pearigee July 28, 2026 15:32

@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 required Node.js engine version in core/common/package.json from >=18 to >=22. There are no review comments, and I have no feedback to provide.

danieljbruce added a commit that referenced this pull request Jul 29, 2026
# Description

Right now we are upgrading Node on client libraries to only support
versions 22+ so open PR tests are naturally failing when trying to run
the tests on Node version 18. While we could remove testing for Node 18
only after we officially drop support for Node 18 to catch specific Node
18 issues in the short term, we create a situation in our CI where our
unit tests will always fail on Node 18 and unit tests for other Node
versions don't run like in
#8988 after the
upgrade. Therefore, we want to remove Node 18 testing right now in order
to keep the tests running for other versions and avoid having a Node 18
unit test that we expect to fail.

The github actions also needed to be updated for the zizmor check.

## Impact

Ensure testing infrastructure is still useful as we do the Node upgrade.
@danieljbruce
danieljbruce enabled auto-merge (squash) July 29, 2026 19:39
@danieljbruce
danieljbruce merged commit d38c05a into main Jul 29, 2026
33 checks passed
@danieljbruce
danieljbruce deleted the node-22-common branch July 29, 2026 19:45
This was referenced Jul 29, 2026
danieljbruce pushed a commit that referenced this pull request Jul 29, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>common: 7.0.0</summary>

##
[7.0.0](common-v6.1.0...common-v7.0.0)
(2026-07-29)


### ⚠ BREAKING CHANGES

* Update common to minimum Node version of 22.
([#8988](#8988))

### Features

* Update common to minimum Node version of 22.
([#8988](#8988))
([d38c05a](d38c05a)),
refs
[#8985](#8985)
</details>

<details><summary>agentidentity: 0.1.0</summary>

## 0.1.0 (2026-07-29)


### Features

* **google/cloud/agentidentity/v1beta:** Onboard a new library
([#9013](#9013))
([acd5989](acd5989))
* **google/cloud/agentidentity/v1:** Onboard a new library
([#9014](#9014))
([b4cef83](b4cef83))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
@release-please release-please Bot mentioned this pull request Jul 29, 2026
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