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

fixes package name override doesn't work #277

Merged
merged 3 commits into from
May 12, 2017

Conversation

favadi
Copy link
Contributor

@favadi favadi commented Dec 8, 2016

Fixed failed tests in #207.

  • option go_package = "github.com/9bug/bug;pb";
  • option go_package = "github.com/9bug/bug-1";

@tmc
Copy link
Collaborator

tmc commented Feb 17, 2017

@favadi thank you for your contribution -- have you signed the CLA? could you include a test case please?

@favadi
Copy link
Contributor Author

favadi commented Feb 21, 2017

I signed the CLA but the PR needs rebase, will do it soon.

@favadi
Copy link
Contributor Author

favadi commented May 4, 2017

@tmc I rebased this PR and add a test case to show the problem.

@tmc tmc merged commit 979be44 into grpc-ecosystem:master May 12, 2017
tamalsaha pushed a commit to gomodules/grpc-gateway that referenced this pull request May 13, 2017
* adds a test case for package overriding

* fixes package name override doesn't work

* sanitizes package name
@favadi favadi deleted the package-name-override branch May 25, 2017 04:03
tamalsaha pushed a commit to gomodules/grpc-gateway that referenced this pull request May 31, 2017
* adds a test case for package overriding

* fixes package name override doesn't work

* sanitizes package name
adasari pushed a commit to adasari/grpc-gateway that referenced this pull request Apr 9, 2020
* adds a test case for package overriding

* fixes package name override doesn't work

* sanitizes package name
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.

2 participants