-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Added useful tip with local addresses #141
Conversation
…to remember your local IP.
Add useful tip with local addresses on start
lgtm at first glace. I will double check over the weekend and (likely) merge this in as is. |
Looks like it's going to repeat its predecessor's fate which had awaited 1 year 3 month and 7 days to be |
@@ -98,6 +101,16 @@ function listen(port) { | |||
+ ' on: '.yellow | |||
+ (protocol + '//' + host + ':' + port).cyan); | |||
|
|||
log('\nAvailable on:'.yellow); | |||
|
|||
for (var dev in ifaces) { |
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.
Do not use for in
loops.
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.
Use Object.keys(ifaces).forEach(function (dev) {
instead.
Some comments for you. |
log('\nAvailable on:'.yellow); | ||
|
||
for (var dev in ifaces) { | ||
ifaces[dev].forEach(function(details){ |
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.
Conform to style here: ifaces[dev].forEach(function (details) {
Are there still any chances of merging or at least adding such a simple function? I'm just curious, nothing personal. 😇 |
Whoops. Didn't get emails for your new commits. In the future comment after committing and I'll be sure to merge it faster. |
Added useful tip with local addresses
Added useful tip with local addresses not to run
ifconfig
command every time you want to recall your local IP.Once again 😉 #60