Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Storage] Delete file and folder (ipfs and chainsafe types) inside a bucket test #2233

Merged
merged 9 commits into from
Jul 26, 2022

Conversation

juans-chainsafe
Copy link
Contributor

closes #2227

@juans-chainsafe juans-chainsafe added the Type: Maintenance Added to issues and PRs when a change is for repository maintenance , such as CI or linter changes. label Jul 22, 2022
@juans-chainsafe juans-chainsafe self-assigned this Jul 22, 2022
@juans-chainsafe juans-chainsafe linked an issue Jul 22, 2022 that may be closed by this pull request
@render
Copy link

render bot commented Jul 22, 2022

@render
Copy link

render bot commented Jul 22, 2022

@render
Copy link

render bot commented Jul 22, 2022

Copy link
Member

@asnaith asnaith left a comment

Choose a reason for hiding this comment

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

@juans-chainsafe Looks great, thanks for this additional test coverage.

I left a few suggestions for nits but otherwise, this looks good to go 😃

juans-chainsafe and others added 6 commits July 22, 2022 15:14
Co-authored-by: Andrew Snaith <asnaith@users.noreply.github.com>
Co-authored-by: Andrew Snaith <asnaith@users.noreply.github.com>
Co-authored-by: Andrew Snaith <asnaith@users.noreply.github.com>
Co-authored-by: Andrew Snaith <asnaith@users.noreply.github.com>
Co-authored-by: Andrew Snaith <asnaith@users.noreply.github.com>
Copy link
Contributor

@tanmoyAtb tanmoyAtb left a comment

Choose a reason for hiding this comment

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

Looks green to go !

@juans-chainsafe juans-chainsafe merged commit 604c9aa into dev Jul 26, 2022
@juans-chainsafe juans-chainsafe deleted the mnt/delete-file-and-folder-storage-test-2227 branch July 26, 2022 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Maintenance Added to issues and PRs when a change is for repository maintenance , such as CI or linter changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Storage] Add delete file and folder inside a bucket test
3 participants