Skip to content

cleanup(pubsub): cleanup BatchWorker loop and Publisher tests#4332

Merged
PhongChuong merged 2 commits intogoogleapis:mainfrom
PhongChuong:comments
Jan 22, 2026
Merged

cleanup(pubsub): cleanup BatchWorker loop and Publisher tests#4332
PhongChuong merged 2 commits intogoogleapis:mainfrom
PhongChuong:comments

Conversation

@PhongChuong
Copy link
Copy Markdown
Collaborator

Followup to @dbolduc 's comments in #4286

@product-auto-label product-auto-label bot added the api: pubsub Issues related to the Pub/Sub API. label Jan 21, 2026
@PhongChuong PhongChuong marked this pull request as ready for review January 21, 2026 23:41
@PhongChuong PhongChuong requested a review from a team January 21, 2026 23:41
dbolduc
dbolduc previously approved these changes Jan 21, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.87%. Comparing base (7653338) to head (6757fe5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4332   +/-   ##
=======================================
  Coverage   94.87%   94.87%           
=======================================
  Files         187      187           
  Lines        7219     7219           
=======================================
  Hits         6849     6849           
  Misses        370      370           

☔ 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.

@PhongChuong
Copy link
Copy Markdown
Collaborator Author

@dbolduc , can you review/approve this PR again? I forgot to run cargo fmt previously.
Thanks.

@PhongChuong PhongChuong enabled auto-merge (squash) January 22, 2026 14:01
@PhongChuong PhongChuong merged commit 8b70794 into googleapis:main Jan 22, 2026
30 checks passed
@PhongChuong PhongChuong deleted the comments branch January 24, 2026 03:05
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.

2 participants