-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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
usb: shell: Remove use of ctx_shell #81395
Conversation
The ctx_shell was only used a single place and in a function that already has a `sh` that would be better to use. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
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.
@Thalley why a draft? This looks like a trivial change.
I always keep PRs in draft until they reach their first CI pass :) |
@jfischer-no please review :) |
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, although I have to bring it back soon.
:( In BT we have #74652 as a step forward to eliminating it |
The ctx_shell was only used a single place and in a function that already has a
sh
that would be better to use.