-
Notifications
You must be signed in to change notification settings - Fork 179
Allow downstream impl of mbed dm
to print help
#751
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
Conversation
The mbed dm subcommand has a complicated invocation. Instead of duplicated the argument parser, I elected to pass the implementation of `mbed dm --help` to the "downsteam implementation" (the python script in mbed-os).
Single line implementation! Love it! 👍 |
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.
@MarceloSalazar Please try and review.
Thanks for looking into this. Now there is some help :) However, as user I'm confused on which command I need to run.
Why do I need to care about "device_management.py" ? My interface is (and should continue to be) Mbed CLI |
@MarceloSalazar That's something that should be changed in device_management.py. |
@MarceloSalazar I'll create a PR to have the
|
@MarceloSalazar Mbed OS PR up: ARMmbed/mbed-os#8104 |
The mbed dm subcommand has a complicated invocation. Instead of
duplicated the argument parser, I elected to pass the implementation
of
mbed dm --help
to the "downsteam implementation" (the pythonscript in mbed-os).
Resolves help issues in #747