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
Improve help for conan remove command #5899
Conversation
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.
Thanks for the contribution!
Please check the comment, the previous was not wrong, the feature is to add that if no argument is passed to --packages
all of them are removed.
conans/client/command.py
Outdated
@@ -994,7 +994,7 @@ def remove(self, *args): | |||
help="Remove only outdated from recipe packages. " | |||
"This flag can only be used with a reference") | |||
parser.add_argument('-p', '--packages', nargs="*", action=Extender, | |||
help="Select package to remove specifying the package ID") | |||
help="Remove all packages of the specified reference") |
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.
This is good, but the previous one is still valid. --packages=xxxxx
will remove that package with the xxxxx ID, and --packages
will remove all packages.
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.
Okay then I think "Remove all packages of the specified reference if no specific package ID is provided" describe the both ?
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.
Yes, good
Thanks! |
Changelog: Fix: Improved
conan remove --help
message for--packages
Docs: Omit
Fix #5495
develop
branch, documenting this one.Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.