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

compose package and service name and regard as fully-qualified service name #236

Merged
merged 13 commits into from
Feb 8, 2020

Conversation

ktr0731
Copy link
Owner

@ktr0731 ktr0731 commented Feb 5, 2020

package is a part of Protocol Buffers, not gRPC.
So, we should not concern about things related to packages.
However, in the perspective of backward-compatibility, we still keep REPL commands related to package.

@ktr0731 ktr0731 self-assigned this Feb 5, 2020
@codecov
Copy link

codecov bot commented Feb 5, 2020

Codecov Report

Merging #236 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #236   +/-   ##
=======================================
  Coverage   80.32%   80.32%           
=======================================
  Files          47       47           
  Lines        2506     2506           
=======================================
  Hits         2013     2013           
  Misses        310      310           
  Partials      183      183

@ktr0731 ktr0731 changed the title treat package and service name as fully-qualified service name compose package and service name and regard as fully-qualified service name Feb 8, 2020
@ktr0731 ktr0731 merged commit 277a4cb into master Feb 8, 2020
@ktr0731 ktr0731 deleted the refactor-spec branch February 8, 2020 04:28
@ktr0731 ktr0731 added the update label Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant