Skip to content

Mark the gRPC broadcast API as deprecated#651

Merged
thanethomson merged 1 commit intomainfrom
thane/v0.38.x/650-deprecate-grpc-api
Apr 5, 2023
Merged

Mark the gRPC broadcast API as deprecated#651
thanethomson merged 1 commit intomainfrom
thane/v0.38.x/650-deprecate-grpc-api

Conversation

@thanethomson
Copy link
Contributor

@thanethomson thanethomson commented Apr 5, 2023

Partially addresses #650.

Changelog entry will be added to the v0.38.x backport.


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

@thanethomson thanethomson requested a review from a team as a code owner April 5, 2023 12:27
Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

LGTM.
You might want to merge this first to main, so when it is finally removed, the git-blame will clearly show the transition "deprecated" --> "removed"

@thanethomson thanethomson changed the base branch from v0.38.x to main April 5, 2023 17:09
@thanethomson thanethomson requested a review from a team as a code owner April 5, 2023 17:09
@thanethomson thanethomson force-pushed the thane/v0.38.x/650-deprecate-grpc-api branch from baac2d7 to 131e20b Compare April 5, 2023 18:10
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@thanethomson thanethomson force-pushed the thane/v0.38.x/650-deprecate-grpc-api branch 2 times, most recently from 50c4755 to 1d17a31 Compare April 5, 2023 18:12
@thanethomson thanethomson changed the title Mark the v0.38.x gRPC broadcast API as deprecated Mark the gRPC broadcast API as deprecated Apr 5, 2023
@thanethomson thanethomson added the backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x label Apr 5, 2023
@thanethomson thanethomson merged commit 66d4aae into main Apr 5, 2023
@thanethomson thanethomson deleted the thane/v0.38.x/650-deprecate-grpc-api branch April 5, 2023 18:22
mergify bot pushed a commit that referenced this pull request Apr 5, 2023
Signed-off-by: Thane Thomson <connect@thanethomson.com>
(cherry picked from commit 66d4aae)
thanethomson added a commit that referenced this pull request Apr 5, 2023
* Deprecate gRPC broadcast API (#651)

Signed-off-by: Thane Thomson <connect@thanethomson.com>
(cherry picked from commit 66d4aae)

* Add changelog entry

Signed-off-by: Thane Thomson <connect@thanethomson.com>

---------

Signed-off-by: Thane Thomson <connect@thanethomson.com>
Co-authored-by: Thane Thomson <connect@thanethomson.com>
thanethomson added a commit that referenced this pull request May 6, 2023
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants