Skip to content

feat: extend upload-firmware with timeout/monitoring/label arguments - #80

Merged
martonilles merged 1 commit into
masterfrom
upload-firmware-options
Aug 26, 2026
Merged

feat: extend upload-firmware with timeout/monitoring/label arguments#80
martonilles merged 1 commit into
masterfrom
upload-firmware-options

Conversation

@martonilles

Copy link
Copy Markdown
Contributor

This way upload timeout, firmware monitoring and labels can be specified on the command line:

upload-firmware --label a --label b --timeout 100 --monitoring ...

#74 #76 #77

@martonilles

Copy link
Copy Markdown
Contributor Author

Extends #79

@qkaiser qkaiser left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM even tho I would maybe split each flag into its own commit

Comment thread onekey_client/cli/firmware_upload.py Outdated
Comment thread onekey_client/cli/firmware_upload.py Outdated
Comment thread onekey_client/client.py
This way upload timeout, firmware monitoring and labels can be specified on
the command line:

upload-firmware --label a --label b --timeout 100 --monitoring ...
@martonilles
martonilles force-pushed the upload-firmware-options branch from 5ba433d to 6128f5f Compare August 26, 2026 17:22
@martonilles
martonilles added this pull request to the merge queue Aug 26, 2026
Merged via the queue into master with commit 308049b Aug 26, 2026
2 checks passed
@martonilles
martonilles deleted the upload-firmware-options branch August 26, 2026 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants