-
Notifications
You must be signed in to change notification settings - Fork 196
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
restructure storage-users upload command #4403
restructure storage-users upload command #4403
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
The full help text shows:
When reading this, I interpret that I can manage uploads in general but only if you go deeper you see that it is to fix upload issues. Maybe to write: |
Co-authored-by: Martin <[email protected]>
we decided to use the texts from 7d76650 |
I was not able to check this but want to highlight a possible thing to be fixed too: From the old outputs:
Should'nt this be: Note that It should look like:
|
Now it is:
|
💥 Acceptance test Core-API-Tests-ocis-storage-6 failed. Further test are cancelled... |
Kudos, SonarCloud Quality Gate passed! |
Description
moves the "uploads" command of the storage-users service into a subsection "maintenance", rewords the descriptions and renames the
ocis storage-users uploads purge
command intoclean
(breaking change).Before:
After:
Before:
After:
Related Issue
Motivation and Context
have a category for a better overview
How Has This Been Tested?
Screenshots (if appropriate):
see description for output
Types of changes
Checklist: