Skip to content

sample(storage-control): add Ruby delete_folder_recursive sample#34703

Open
nidhiii-27 wants to merge 9 commits into
mainfrom
feat/add-ruby-delete-folder-recursive
Open

sample(storage-control): add Ruby delete_folder_recursive sample#34703
nidhiii-27 wants to merge 9 commits into
mainfrom
feat/add-ruby-delete-folder-recursive

Conversation

@nidhiii-27

Copy link
Copy Markdown
Contributor

Adds a Ruby code sample demonstrating hierarchical namespace recursive folder delete.

Fixes: b/530059259

[Generated-by: AI]

@nidhiii-27 nidhiii-27 added ai-generated Generated by AI storage-sample-architect Generated by storage-sample-architect skill labels Jul 2, 2026
Updates compose file sample to support deleting source objects optionally. Fixes b/441557254

[Generated-by: AI]
Resolve comments in files_test.rb by adding assertions on source files and testing error scenarios.

[Generated-by: AI]
Adds a Ruby code sample demonstrating hierarchical namespace recursive folder delete.

Fixes: b/530059259

[Generated-by: AI]
Redefines StorageDownloadCommand#execute_once to skip chunk processing and offset modification for non-2xx status codes (e.g. 503 errors). This prevents range headers from being sent incorrectly on retry attempts.

[Generated-by: AI]
@nidhiii-27 nidhiii-27 force-pushed the feat/add-ruby-delete-folder-recursive branch from 5bc8902 to d751a7b Compare July 2, 2026 08:47
@nidhiii-27 nidhiii-27 marked this pull request as ready for review July 2, 2026 09:39
@nidhiii-27 nidhiii-27 requested review from a team and yoshi-approver as code owners July 2, 2026 09:39
@snippet-bot

snippet-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 1 region tag.

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

Reverts a monkey patch in StorageDownloadCommand that was causing regressions in acceptance tests. Also sorts custom contexts in the storage_get_object_contexts sample to ensure consistent output and fix flaky tests. Additionally fixes a bug in the delete_folder_recursive sample entry point.

[Generated-by: AI]
Fixed acceptance tests in google-cloud-storage that were broken by incorrect bucket references and restricted names.

[Generated-by: AI]
@bajajneha27

Copy link
Copy Markdown
Contributor

Since this is just adding samples, the conventional commit message should be either chore or sample but not feat . Changes with feat will appear in CHANGELOG and this shouldn't be part of it.

@bajajneha27 bajajneha27 changed the title feat(storage-control): add Ruby delete_folder_recursive sample sample(storage-control): add Ruby delete_folder_recursive sample Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Generated by AI storage-sample-architect Generated by storage-sample-architect skill

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants