Skip to content

Added option to print the unix like command itself#149

Closed
geekflyer wants to merge 1 commit intoshelljs:masterfrom
geekflyer:master
Closed

Added option to print the unix like command itself#149
geekflyer wants to merge 1 commit intoshelljs:masterfrom
geekflyer:master

Conversation

@geekflyer
Copy link
Copy Markdown

This is similar to the option set -o verbose in regular shell scripts. Partially related to #8.
Without this option some commands are completely silent, e.g. mkdir('somedir') would just create a directory silently. With the logCmd option it will additionally print mkdir somedir, to give the user an idea what the script is actually doing.

@nfischer
Copy link
Copy Markdown
Member

This could be a cool option to add as part of a set command. @geekflyer would you have any interest in writing a set command to act as a wrapper for config variables? If we can get that out for config.fatal, then we might be able to add this as well.

If it uses a set() interface, it'll probably be more intuitive to Bash scripters, and might help tame the docs section (which is rather lengthy).

@nfischer
Copy link
Copy Markdown
Member

nfischer commented Feb 9, 2016

Thanks so much for the PR @geekflyer! I think we'll have to close this, though, since this has been incorporated in #329

@nfischer nfischer closed this Feb 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants