Skip to content
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

Allow list filters to work with usb list command #564

Merged

Conversation

JamesHagerman
Copy link
Contributor

@JamesHagerman JamesHagerman requested a review from busticated April 8, 2020 06:58
Copy link
Contributor

@busticated busticated left a comment

Choose a reason for hiding this comment

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

looks good 👍

let's add tests here

src/cmd/usb.js Outdated Show resolved Hide resolved
@technobly
Copy link
Member

Would be good to add some notes in this PR for how this new feature works. Or a link to a Docs PR.

…tests for those filters. Adding unit tests and e2e tests to both Cloud List and USB List functionality.
@JamesHagerman JamesHagerman force-pushed the ch51710/allow-list-filters-to-work-with-usb-list branch from 2c1cc51 to de0b32d Compare April 23, 2020 09:01
Copy link
Contributor

@busticated busticated left a comment

Choose a reason for hiding this comment

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

🎉🎉🎊😍👍

src/lib/utilities.js Show resolved Hide resolved
test/e2e/cloud.e2e.js Show resolved Hide resolved
@JamesHagerman JamesHagerman merged commit 915f933 into master Apr 23, 2020
@JamesHagerman JamesHagerman deleted the ch51710/allow-list-filters-to-work-with-usb-list branch April 23, 2020 18:01
@JamesHagerman
Copy link
Contributor Author

@technobly For the record, usage is documented in the output of particle usb list --help. It's the same usage as particle list!

Param filter can be: online, offline, a platform name (photon, electron, etc), a device ID or name

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