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

Add go_package statements for the Go-supported WKTs. #1361

Closed
wants to merge 1 commit into from
Closed

Add go_package statements for the Go-supported WKTs. #1361

wants to merge 1 commit into from

Conversation

dsymonds
Copy link

This instructs the Go protobuf code generator how to generate code in the right
place and generate correct import statements. The design for this feature is
described in golang/protobuf#139.

This is an integration of a Google-internal change.

@xfxyjwf
Copy link
Contributor

xfxyjwf commented Mar 31, 2016

@pherl Jisi has a pull request integrating all internal changes to github:
#1365

It seems to contain this go_package change as well.

@dsymonds
Copy link
Author

It looks like #1365 got abandoned because of some other problems. Can we merge this one now? It's causing a bit of pain elsewhere.

@grpc-kokoro
Copy link

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

This instructs the Go protobuf code generator how to generate code in the right
place and generate correct import statements. The design for this feature is
described in golang/protobuf#139.

This is an integration of a Google-internal change.
@xfxyjwf
Copy link
Contributor

xfxyjwf commented Apr 11, 2016

ok to test

@Globegitter
Copy link

Is there anything still blocking this?

@dsymonds
Copy link
Author

@xfxyjwf, please merge?

@liujisi
Copy link
Contributor

liujisi commented Apr 28, 2016

I think the other integration has already been merged. See https://github.com/google/protobuf/blob/master/src/google/protobuf/any.proto

@dsymonds
Copy link
Author

Indeed, thanks.

@dsymonds dsymonds closed this Apr 28, 2016
dstore-dbap pushed a commit to dstore-io/dstore-sdk-golang that referenced this pull request Jun 21, 2016
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.

6 participants