-
Notifications
You must be signed in to change notification settings - Fork 297
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
wspb.go uses deprecated protobuf package #311
Comments
Oh very nice! Will definitely do in the next major release as it would be a breaking change. Though perhaps this also suggests protobuf helpers don't belong in the core package and should be published elsewhere? |
Yes, moving the helpers out e.g. into an example that can be copied or into a separate module would be good to avoid transitive dependencies. |
The library we're currently using for protobufs is deprecated. Doesn't belong in the library core anyway. Closes #311
Done in dev. |
Is this a joke? Why wspb - a public package got removed?! |
I love the idea of the protobuf helpers in wspb.go, but when trying to use them I noticed that wspb.go imports
github.com/golang/protobuf/proto
. Since this module is deprecated, this helper can't easily be used in a codebase using the newer protobuf library (google.golang.org/protobuf
).From https://github.com/golang/protobuf:
It would be wonderful if this import could be switched out to use the newer module 😀
The text was updated successfully, but these errors were encountered: