-
Notifications
You must be signed in to change notification settings - Fork 326
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
T5944: Fix reboot in arg #2821
T5944: Fix reboot in arg #2821
Conversation
Backport required |
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.
It is probably better to change the command line argument in the script to
--reboot-in
since argparse will correctly read this as args.reboot_in
, and this is the normalized form when this script is moved to use opmode.py.
Fix the arg for the `reboot in x` command The current arg is `--reboot_in [Minutes ...]` The expected arg is `--reboot-in [Minutes ...]`
@jestabro Done! |
@Mergifyio backport sagitta |
✅ Backports have been created
|
Change Summary
Fix the arg for the
reboot in x
commandThe actual arg is
--reboot_in [Minutes ...]
Types of changes
Related Task(s)
Related PR(s)
Component(s) name
reboot, power
Proposed changes
How to test
Before the fix:
After the fix:
Smoketest result
Checklist: