feat(storage): add sample for hierarchical namespace recursive delete - #4434
Conversation
Adds a Node.js code sample demonstrating hierarchical namespace recursive folder delete. Fixes: b/530059535 [Generated-by: AI]
Add helpful log statements.
…ests Re-enable recursive delete folder integration tests and resolve GCS billing project limitations. [Generated-by: AI]
Remove skip comment for 'should delete a folder recursively' integration test. [Generated-by: AI]
…folder sample [Generated-by: AI]
product-auto-label
Bot
added
api: storage
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new sample script deleteFolderRecursive.js to recursively delete folders, updates renameFolder.js to strip billing project headers, bumps the @google-cloud/storage-control dependency, and updates system tests to explicitly pass the project ID to the Storage client. It also enables the folder renaming test and adds a test for recursive deletion. Regarding the feedback, there is an issue in deleteFolderRecursive.js where a failed operation is incorrectly logged as successful; the code should check for op.error and throw an error if the operation fails.
…iew comments [Generated-by: AI]
[Generated-by: AI]
|
Here is the summary of changes. You are about to add 1 region tag.
This comment is generated by snippet-bot.
|
[Generated-by: AI]
angelcaamal
left a comment
angelcaamal
left a comment
There was a problem hiding this comment.
The code looks solid to me.
3 participants
Draft PR for hierarchical namespace recursive delete folder sample.
[Generated-by: AI]