-
-
Notifications
You must be signed in to change notification settings - Fork 50
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 File Parameter #56
base: master
Are you sure you want to change the base?
Conversation
console.log(chalk.yellow('Opening file: ' + file)) | ||
hostile.setFile(file); | ||
} | ||
|
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.
Checks for existence of param independent of command.
@@ -30,6 +37,8 @@ function help () { | |||
remove [domain] Remove a domain from the hosts file | |||
load [file] Load a set of host entries from a file | |||
unload [file] Remove a set of host entries from a file | |||
|
|||
optional --file parameter to all commands to allow for manipulating other hosts files in non standard locations |
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.
Just adding to the help prompt.
{ | ||
exports.HOSTS = filePath | ||
} | ||
|
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.
As the app runs per instance this should be reset every time it's run unless I'm missing something.
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.
Just added comments
Added a --file parameter to all commands to allow for specifying a hosts file location. Sometimes I want to edit a different file than the system one, especially when dealing with docker.