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

Looking for your Tessel... #340

Closed
Student007 opened this issue Sep 25, 2015 · 2 comments
Closed

Looking for your Tessel... #340

Student007 opened this issue Sep 25, 2015 · 2 comments

Comments

@Student007
Copy link
Member

As discussed here I open as recommended this issue.

I would prefer to replace
ogs.info('Looking for your Tessel...');
with something like this (e.g.):

if(opts.lan) {
  logs.info('Searching nearby LAN (netmask) for Tessels...');
} else if(opts.usb) {
  logs.info('Searching your Tessel connected to USB...');
} else {
  logs.info('Searching USB and nearby LAN (netmask) for Tessels...');
} 

Of cause there would be a better wording but I feel the current wording is only fitting if someone owns a single Tessel.
Also if you are on a HackEvent, you would use the same WLAN what means you will find multiple Tessels. Following the given wording, I think it is finding to much Tessels. It would match if it only would find Tessels are provisioned with your computer.

@Student007 Student007 mentioned this issue Sep 25, 2015
@johnnyman727
Copy link
Contributor

I think Looking for your Tessel is more appropriate because even if there are multiple Tessels around, the CLI is trying to find the single Tessel that you want to operate on (since you can only operate on one at a time). Users can be passing in other options (--usb or --lan or --name) to help the CLI narrow down all the discovered Tessels to the single one they want to use.

@Student007
Copy link
Member Author

@johnnyman727 close because finally you ever be right 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants