-
Notifications
You must be signed in to change notification settings - Fork 126
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
Adding commandline options to list boards and ports #27
Conversation
@makenai thank you for this! These are really nice ideas/additions. I have some feedback for this PR, but need a little time to write it up. Stay tuned! |
You can list available USB ports programmatically using the the `listPorts` method: | ||
|
||
```javascript | ||
Avrgurl.listPorts(function(err, ports) { |
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.
spelling of Avrgirl
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.
Doh!
Thanks for the feedback, Suz! Give me a few days and I'll address the issues. |
…ome test boilerplate and tests for board detection.
3ec0a3e
to
c3b1d60
Compare
Sorry it took me a bit longer than I thought to get back to this. I've incorporated your suggested changes and rebased against master. How does this look? |
@noopkat Patch up to preserve the exit status - thanks for explaining! |
Thanks @makenai! Merged and will publish soon with some other additions (like CI / new styleguide stuff and readme changes). Just a small note - for now I have consolidated the I really appreciate these contributions though, so thanks again 👏 |
Thank you, Susan. |
Closes #1
Added commandline options to list boards and ports. Started adding some test boilerplate to build upon and tests for board detection.
Let me know if any of the output or refactoring isn't up to spec and I can fix it up.