Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Sep 11, 2015

Works around google/protorpc#12.

Fixes: #1133.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 11, 2015
@tseaver
Copy link
Contributor Author

tseaver commented Sep 11, 2015

@dhermes PTAL: google/protorpc#12 blocks Travis for us.

@tseaver
Copy link
Contributor Author

tseaver commented Sep 11, 2015

Obsoleted by @craigcitro's quick release of 0.11.1 with a backward-compatibility alias.

@tseaver tseaver closed this Sep 11, 2015
@tseaver tseaver deleted the 1133-workaround_protorpc_0.11.0 branch September 11, 2015 15:56
@dhermes
Copy link
Contributor

dhermes commented Sep 16, 2015

Thanks @craigcitro!

Is protorpc becoming compatible with proto3?

@craigcitro
Copy link
Contributor

no one's actively working on it, though i'm happy to review patches if you're interested.

TBH, though, i'm not quite sure what you mean by "proto3-compatible" -- the main way to use protorpc doesn't read .proto files, and the wire protocol is still the same. and, all the proto3 constructs have proto2 equivalents. was there something you wanted to do with it?

@dhermes
Copy link
Contributor

dhermes commented Sep 16, 2015

Nope was just curious. Thanks. (FWIW I didn't think my question all the way and your answer makes a lot of sense)

parthea pushed a commit that referenced this pull request Nov 24, 2025
There are some bugs in this implementation that stop generation
in some APIs and result in incorrect snippets getting generated
for others. Back out this feature for now so we can move forward
with enabling snippetgen for all APIs.
parthea pushed a commit that referenced this pull request Nov 24, 2025
I accidentally merged #1134 without updating the golden files for snippetgen. This PR updates the golden files and marks the 'snippetgen' check required.
parthea pushed a commit that referenced this pull request Nov 24, 2025
🤖 I have created a release *beep* *boop*
---


## [0.62.0](googleapis/gapic-generator-python@v0.61.0...v0.62.0) (2022-01-28)


### Features

* adds REST server-streaming support. ([#1120](googleapis/gapic-generator-python#1120)) ([812cf3e](googleapis/gapic-generator-python@812cf3e))
* expose extended operations annotations within generator ([#1145](googleapis/gapic-generator-python#1145)) ([e7bee70](googleapis/gapic-generator-python@e7bee70))


### Bug Fixes

* add special method parameters to set of reserved module names ([#1168](googleapis/gapic-generator-python#1168)) ([8078961](googleapis/gapic-generator-python@8078961))
* fix snippetgen golden file ([#1170](googleapis/gapic-generator-python#1170)) ([13b2028](googleapis/gapic-generator-python@13b2028))
* **snippetgen:** remove special handling for resource paths ([#1134](googleapis/gapic-generator-python#1134)) ([4ea2d11](googleapis/gapic-generator-python@4ea2d11))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants