Skip to content

Conversation

@alice-sawatzky
Copy link

Add ListFolders() function to all storage modules. Refactor modules to use an iterator to reduce redundant code between ListObjects() and ListFolders().

The goal is to support a chartmuseum endpoint described in helm/chartmuseum#360.

This is my first contribution to a Go project so feedback and patience is appreciated!

Progress:
[x] alibaba
[x] amazon
[x] baidu
[x] etcd
[x] google
[x] local
[x] microsoft
[x] netease
[x] openstack
[x] oracle
[x] tencent
[x] update Backend interface
[x] beef up tests a bit

@alice-sawatzky alice-sawatzky mentioned this pull request Nov 18, 2020
13 tasks
won't work until we add ListFolders to the Backend interface
@alice-sawatzky
Copy link
Author

I've rebased to the latest main, the merge conflict didn't appear to be relevant.

@alice-sawatzky
Copy link
Author

hello! I've been on leave, just coming back, is there any interest in moving forward with this feature? I realize I left this hanging for a long time, but the conflicts look pretty manageable considering how long it's been

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.

1 participant