-
Notifications
You must be signed in to change notification settings - Fork 15.4k
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
GA fixes #1847
GA fixes #1847
Conversation
This brings us more into line with other langauges, and makes it more obvious when we are requiring protobuf generated code.
Ruby: generated foo.proto -> foo_pb.rb instead of foo.rb.
[ci skip] Change-Id: Ia0e0bf3bb6bd24633dc8f294318bb20ac837f8b9
Add a script to build protoc zip packages.
Ruby: translate package names from snake_case -> PascalCase.
We need to pick up some of the last-minute Ruby fixes.
We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm. |
Seems cpp_distcheck is still failing. Can you update Makefile.am to remove the deleted ruby files? |
Done, PTAL. |
LGTM |
This is a merge of a master, to pick up a few fixes from after the branch was cut.
Review to @xfxyjwf.