Skip to content

chore(test-utils): revert making google-test-utils a private workspace package - #9548

Merged
shivanee-p merged 4 commits into
mainfrom
shivaneep-revert-test-utils
Oct 6, 2026
Merged

shivanee-p merged 4 commits into
mainfrom
shivaneep-revert-test-utils

Conversation

@shivanee-p

@shivanee-p shivanee-p commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Reverts #9525 (be25b6fa1a6c6c90337ecad8afdcbcaeb477ded2) to keep google-test-utils (core/test-utils) as a published package managed by Release Please, and updates consumer packages to depend on the published ^0.2.0 release

  • This is a temporary fix while we investigate if we can use pnpm instead of npm

@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 makes the core/test-utils package public, registers it in the release-please configuration, and updates its usage in the pubsub and spanner packages. Feedback points out that the dependency version for google-test-utils in both packages should be updated to workspace:^0.2.0 to match the workspace version and prevent installation failures. Additionally, the Cloud Build filters for spanner should be updated to use the prefix-only syntax ...{./handwritten/spanner} to avoid building unnecessary downstream dependents.

Comment thread handwritten/spanner/package.json Outdated
Comment thread handwritten/pubsub/package.json Outdated
Comment thread handwritten/spanner/cloudbuild.yaml Outdated
Comment thread handwritten/spanner/cloudbuild-regular-sessions.yaml Outdated
@shivanee-p shivanee-p changed the title Revert "fix(test-utils): make google-test-utils a private workspace package" fix(test-utils): revert making google-test-utils a private workspace package Oct 6, 2026
@shivanee-p
shivanee-p force-pushed the shivaneep-revert-test-utils branch from a9cfe58 to afee093 Compare October 6, 2026 22:51
@shivanee-p
shivanee-p marked this pull request as ready for review October 6, 2026 22:51
@shivanee-p
shivanee-p requested review from a team as code owners October 6, 2026 22:51
@shivanee-p
shivanee-p requested review from quirogas and removed request for a team October 6, 2026 22:52

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

LGTM as long as tests pass.

@github-actions
github-actions Bot requested a review from danieljbruce October 6, 2026 22:55
@shivanee-p shivanee-p changed the title fix(test-utils): revert making google-test-utils a private workspace package chore(test-utils): revert making google-test-utils a private workspace package Oct 6, 2026
@shivanee-p
shivanee-p merged commit 34f8d61 into main Oct 6, 2026
47 checks passed
@shivanee-p
shivanee-p deleted the shivaneep-revert-test-utils branch October 6, 2026 23:18
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