Skip to content

Remove usage of deprecated methods.#4076

Merged
martinmr merged 1 commit intomasterfrom
martinmr/grpc-deprecated
Sep 27, 2019
Merged

Remove usage of deprecated methods.#4076
martinmr merged 1 commit intomasterfrom
martinmr/grpc-deprecated

Conversation

@martinmr
Copy link
Copy Markdown
Contributor

@martinmr martinmr commented Sep 27, 2019

ErrorDesc in the grpc is deprecated. It's been replaced by the
recommended methods.


This change is Reviewable

ErrorDesc in the grpc is deprecated. It's been replaced by the
recommended methods.
@martinmr martinmr requested a review from a team September 27, 2019 01:16
Copy link
Copy Markdown

@hackerone-code hackerone-code Bot left a comment

Choose a reason for hiding this comment

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

✅ A review job has been created and sent to the PullRequest network.


@martinmr you can click here to see the review status or cancel the code review job.

Copy link
Copy Markdown
Contributor

@manishrjain manishrjain left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Copy Markdown

@hackerone-code hackerone-code Bot left a comment

Choose a reason for hiding this comment

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

Good change. Should ensure there is no other similar usages that need updating in the codebase.


Reviewed with ❤️ by PullRequest

@martinmr martinmr merged commit 7572578 into master Sep 27, 2019
@martinmr martinmr deleted the martinmr/grpc-deprecated branch September 27, 2019 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants