Skip to content
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

Join grpc proto files together (CAS-308) #328

Merged
merged 1 commit into from
Jul 20, 2020
Merged

Conversation

jkryl
Copy link
Contributor

@jkryl jkryl commented Jul 16, 2020

Originally they were split so that we could generate common type
definitions for jsonrpc and grpc. That's not needed now.

@jkryl jkryl requested a review from gila July 16, 2020 15:40
@jkryl jkryl self-assigned this Jul 16, 2020
@auto-assign auto-assign bot requested review from tiagolobocastro and tjoshum July 16, 2020 15:41
@jkryl jkryl force-pushed the feature/one-proto branch 2 times, most recently from d48e12c to 78a9b79 Compare July 16, 2020 17:49
@jkryl jkryl requested a review from blaisedias July 20, 2020 10:04
Copy link
Contributor

@blaisedias blaisedias left a comment

Choose a reason for hiding this comment

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

Looka okay to me.

Originally they were split so that we could generate common type
definitions for jsonrpc and grpc. That's not needed now.

Resolves: CAS-308
@jkryl jkryl force-pushed the feature/one-proto branch from 78a9b79 to 91ebe5a Compare July 20, 2020 13:19
@jkryl jkryl merged commit 91ebe5a into develop Jul 20, 2020
@jkryl jkryl deleted the feature/one-proto branch July 20, 2020 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants