Skip to content

chore(java-accessapproval): remove extra generated samples#12560

Merged
zhumin8 merged 1 commit intomainfrom
rm-stale-samples
Apr 1, 2026
Merged

chore(java-accessapproval): remove extra generated samples#12560
zhumin8 merged 1 commit intomainfrom
rm-stale-samples

Conversation

@zhumin8
Copy link
Copy Markdown
Contributor

@zhumin8 zhumin8 commented Apr 1, 2026

Remove extra generated samples under java-accessapproval/google-cloud-accessapproval/samples/snippets/generated.
The generated samples maintained by the generator are located in google-cloud-java/java-accessapproval/samplessnippets/generated.

These deleted samples were introduced in commit. From the commit message it seems to meant for short term testing and is accidentally left in the code base:
chore: include java generated samples testing recent changes in gapic-generator-java 2.7.0 to write generated samples to files

@zhumin8 zhumin8 requested a review from a team as a code owner April 1, 2026 20:09
@snippet-bot
Copy link
Copy Markdown

snippet-bot bot commented Apr 1, 2026

Here is the summary of changes.

You are about to delete 21 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

Copy link
Copy Markdown
Contributor

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

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 removes a comprehensive set of auto-generated Java sample files for the Access Approval V1 API, including both synchronous and asynchronous implementations for managing approval requests and settings. I have no feedback to provide.

@zhumin8 zhumin8 requested a review from blakeli0 April 1, 2026 20:11
@zhumin8 zhumin8 requested review from a team and lqiu96 April 1, 2026 20:11
@zhumin8 zhumin8 enabled auto-merge (squash) April 1, 2026 20:11
@zhumin8 zhumin8 merged commit 4e18407 into main Apr 1, 2026
107 checks passed
@zhumin8 zhumin8 deleted the rm-stale-samples branch April 1, 2026 20:52
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.

2 participants