-
Notifications
You must be signed in to change notification settings - Fork 189
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
[full-ci] Assimilate clean command into sessions #9041
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. |
Signed-off-by: jkoberg <[email protected]>
1d216bc
to
4941620
Compare
Signed-off-by: jkoberg <[email protected]>
Signed-off-by: jkoberg <[email protected]>
d4c1721
to
4b6d1f6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pls add the change also to the readme, the current session commands are documented and would now no longer be valid.
Signed-off-by: jkoberg <[email protected]>
5fab4c8
to
62b1286
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from a docs pov
Quality Gate passedIssues Measures |
[full-ci] Assimilate clean command into sessions
Assimilates the
clean
command intosessions
.Fixes #9003
Changes
ocis storage-users uploads list
got deprecated (it was before but now it prints it)ocis storage-users uploads clean
got deprecatedocis storage-users uploads sessions
got new parametersclean
cleans listed uploads. It is recommended to run withoutclean
flag first to double check which uploads will be removedhas-virus
allows filtering for infected filesocis storage-users uploads sessions --json
did not output proper json, now it doesocis storage-users upload sessions
info string (it prints when run without--json
) was completely bonkedExamples