Skip to content

Conversation

@suzmue
Copy link
Collaborator

@suzmue suzmue commented Jan 15, 2026

This is the first step in simplifying the Publisher client creation and renames the Client and associated builders.

For #4282

@suzmue suzmue requested a review from a team as a code owner January 15, 2026 00:07
@product-auto-label product-auto-label bot added the api: pubsub Issues related to the Pub/Sub API. label Jan 15, 2026
@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.80%. Comparing base (b03561b) to head (e0c5b02).
⚠️ Report is 31 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4283   +/-   ##
=======================================
  Coverage   94.80%   94.80%           
=======================================
  Files         187      187           
  Lines        7194     7194           
=======================================
  Hits         6820     6820           
  Misses        374      374           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

coryan
coryan previously approved these changes Jan 15, 2026
Copy link
Collaborator

@coryan coryan left a comment

Choose a reason for hiding this comment

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

SGTM. I think we can bikeshed the names once we are closer to "not preview".

@suzmue suzmue changed the title feat(pubsub): Rename Client to BasePublisher impl(pubsub): Rename Client to BasePublisher Jan 16, 2026
@suzmue suzmue force-pushed the feat/pubsub-rename-client-to-basepublisher branch from 8b6fa00 to e0c5b02 Compare January 20, 2026 18:58
Copy link
Member

@dbolduc dbolduc left a comment

Choose a reason for hiding this comment

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

refactor(pubsub)!: ... ?

I think we would include this change in our release notes.

@suzmue
Copy link
Collaborator Author

suzmue commented Jan 21, 2026

refactor(pubsub)!: ... ?

I think we would include this change in our release notes.

Right it is released. Updated.

@suzmue suzmue changed the title impl(pubsub): Rename Client to BasePublisher refactor(pubsub)!: Rename Client to BasePublisher Jan 21, 2026
@suzmue suzmue merged commit 918a8f8 into googleapis:main Jan 21, 2026
30 checks passed
@suzmue suzmue deleted the feat/pubsub-rename-client-to-basepublisher branch January 21, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsub Issues related to the Pub/Sub API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants