Skip to content

fix: resolve collision with reserved words in samples - #17912

Merged
parthea merged 1 commit into
mainfrom
avoid-reserved-words-samples
Jul 27, 2026
Merged

fix: resolve collision with reserved words in samples#17912
parthea merged 1 commit into
mainfrom
avoid-reserved-words-samples

Conversation

@parthea

@parthea parthea commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Towards b/537730936

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the GAPIC generator to use the client method name instead of the raw RPC name when generating sample method names. This ensures that Python-specific method name overrides (such as appending an underscore to reserved keywords like import) are correctly reflected in the generated samples. A new Import RPC and its corresponding golden test files have been added to verify this behavior. I have no feedback to provide as there are no review comments.

@parthea
parthea force-pushed the avoid-reserved-words-samples branch 3 times, most recently from 44f7403 to 973dd75 Compare July 27, 2026 18:21
@parthea parthea self-assigned this Jul 27, 2026
@parthea
parthea force-pushed the avoid-reserved-words-samples branch from 973dd75 to 98c5ddb Compare July 27, 2026 18:30
@parthea
parthea force-pushed the avoid-reserved-words-samples branch from 98c5ddb to a68ec1c Compare July 27, 2026 19:02
@parthea
parthea marked this pull request as ready for review July 27, 2026 19:32
@parthea
parthea requested a review from a team as a code owner July 27, 2026 19:32
@snippet-bot

snippet-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown

Here is the summary of possible violations 😱

Details

There are 2 possible violations for not having product prefix.

The end of the violation section. All the stuff below is FYI purposes only.


Here is the summary of changes.

You are about to add 2 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@parthea parthea assigned daniel-sanche and unassigned parthea Jul 27, 2026
@parthea
parthea merged commit 588cda9 into main Jul 27, 2026
99 of 100 checks passed
@parthea
parthea deleted the avoid-reserved-words-samples branch July 27, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants