馃摑 Description:
Currently, the CLI requires a firmware file as a positional argument:
parser.add_argument('file')
This causes issues when running operations that do not need a firmware file, such as:
- Reading chip ID
- Erasing flash memory
馃挕 Proposal:
Update the parser to make the file argument optional:
parser.add_argument('file', nargs='?', default=None, help='Firmware file to read/write')
This allows more flexibility in usage and better support for scripts and automation workflows that don鈥檛 always involve flashing.
馃攳 Context:
I'm using this tool as a submodule and needed this change to support a more dynamic flashing sequence via a Flask-based controller. Happy to open a PR if maintainers agree this direction makes sense!
馃摑 Description:
Currently, the CLI requires a firmware file as a positional argument:
parser.add_argument('file')This causes issues when running operations that do not need a firmware file, such as:
馃挕 Proposal:
Update the parser to make the file argument optional:
parser.add_argument('file', nargs='?', default=None, help='Firmware file to read/write')This allows more flexibility in usage and better support for scripts and automation workflows that don鈥檛 always involve flashing.
馃攳 Context:
I'm using this tool as a submodule and needed this change to support a more dynamic flashing sequence via a Flask-based controller. Happy to open a PR if maintainers agree this direction makes sense!