-
Notifications
You must be signed in to change notification settings - Fork 1.1k
docs: update Pub/Sub publisher docstring example #6100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@chingor13 Do you know why the test kept failing? |
|
It looks like a flakey spanner JDBC test. @kolea2, @olavloite might know more. |
|
Sorry about that, we're tracking #6075 to fix this. I've rerun the build. |
...le-cloud-clients/google-cloud-pubsub/src/main/java/com/google/cloud/pubsub/v1/Publisher.java
Outdated
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #6100 +/- ##
============================================
+ Coverage 47.38% 47.4% +0.01%
- Complexity 27180 27228 +48
============================================
Files 2523 2523
Lines 274580 274830 +250
Branches 31380 31391 +11
============================================
+ Hits 130118 130289 +171
- Misses 134852 134928 +76
- Partials 9610 9613 +3
Continue to review full report at Codecov.
|
Fixes #3920
Missed the docstring in a previous PR #4120 that addressed the same issue.