-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[commands] New conan require
#19457
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
[commands] New conan require
#19457
Conversation
|
Pending issues/discussions:
|
This will be a given if using
No, better only .py |
…d. Improved conan dep UX. Tests OK
dep and initdep
memsharded
left a comment
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.
Looking good
|
@czoido @memsharded I'll open another PR with the full |
memsharded
left a comment
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.
Look good to me.
Maybe my only issue is that conan dep as a command still doesn't fully click. It is a bit different to other commands that are more verb-like conan create/install/export/inspect/list..., the closest one would be conan editable add/remove and conan remote add/remove.
I can't think of a better name though, so all good for me.
Changelog: Feature: Add
conan requirecommand to add/remove requirements to/from your local conanfile.Docs: conan-io/docs#4354