-
Notifications
You must be signed in to change notification settings - Fork 104
refactor(pubsub)!: Rename Client to BasePublisher #4283
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
refactor(pubsub)!: Rename Client to BasePublisher #4283
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 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. 🚀 New features to boost your workflow:
|
coryan
left a comment
There was a problem hiding this 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".
8b6fa00 to
e0c5b02
Compare
dbolduc
left a comment
There was a problem hiding this 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.
Right it is released. Updated. |
This is the first step in simplifying the Publisher client creation and renames the Client and associated builders.
For #4282