Skip to content

Conversation

busunkim96
Copy link
Contributor

No description provided.

@busunkim96 busunkim96 requested a review from a team as a code owner October 27, 2021 19:51
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 27, 2021
@busunkim96 busunkim96 marked this pull request as draft October 27, 2021 20:21
@busunkim96 busunkim96 marked this pull request as ready for review October 29, 2021 20:39
Copy link
Contributor

@software-dov software-dov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. It would be good to have a test for options,Options for default behavior and disabling based on old-naming, but I'm not going to block merge on that.

@busunkim96 busunkim96 merged commit e46f443 into master Oct 29, 2021
@busunkim96 busunkim96 deleted the busunkim96-patch-1 branch October 29, 2021 22:04
gcf-merge-on-green bot pushed a commit that referenced this pull request Nov 2, 2021
busunkim96 added a commit that referenced this pull request Nov 8, 2021
vam-google pushed a commit that referenced this pull request Nov 8, 2021
* Revert "test: add tests for autogen-snippets option (#1055)"

This reverts commit 185ecc7.

* Revert "feat: generate code snippets by default (#1044)"

This reverts commit e46f443.
busunkim96 added a commit that referenced this pull request Feb 3, 2022
Enable snippetgen for the default (non-Ads) templates.

This reverts commit 8bdb709 (which was a revert of #1044 and #1055).

I've checked that the changes are OK (don't break generation for any APIs) by creating a [tag](https://github.com/googleapis/gapic-generator-python/commits/v0.62.0b1) and running the [presubmit](https://critique.corp.google.com/cl/424921742).
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants