-
Notifications
You must be signed in to change notification settings - Fork 8
Move CLI to commander #13
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
Bonus, check out this lovely notbook at notbook in ~/git/nodebots-interchange on commander
$ interchange -h
Usage: interchange [options] [command]
Commands:
list|l Lists all of the available firmwares
ports|p [options] Lists all of the attached boards and their ports
read|r [options] Read firware info from port
install|i [options] [firmware] Install specified firmware to board
Options:
-h, --help output usage information
-V, --version output the version number
notbook at notbook in ~/git/nodebots-interchange on commander
$ interchange install -h
Usage: install|i [options] [firmware]
Install specified firmware to board
Options:
-h, --help output usage information
-a, --board <board> Type of board/AVR
-p, --port <port> Serial port board is attached to
-f, --firmata Install firmata version of firmware
-i, --address <address> Specify I2C address |
Ooh nice one. Will check out your branch shortly and have a look at the code On Wed, Nov 11, 2015, 12:55 Derek Wheelden notifications@github.com wrote:
|
bin/cli.js
Outdated
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.
typo - "firmware"
Looks pretty good other than those little nits which were largely mine anyway ;P After that's done I'll merge it. |
Move CLI to commander - thanks to @Frxnz
I ended up down kind of a weird road on this before coming back to reality. "Interchange" and the CLI itself are in separate files now. This is a result of thinking I needed to do something with commander that ended up being wrong. If you're not happy with this I can revert that portion.
However, I decided to commit it this way because it opens up the possibility of using interchange programmatically in the future. I tested as much as I could and everything seems fine.