Node-bunyan: Suppress "details" in CLI `--output short` format

Created on 5 May 2014  路  8Comments  路  Source: trentm/node-bunyan

I'd like to suppress the "details" output when running the CLI in --output short format. Would you be interested in changing the output for this format? Or would you be interested in adding an option --no-details?

needstriage Type-Improvement Component-CLI

Most helpful comment

I'd love to see this merged as well!

All 8 comments

:+1:

PR for this: #143 @aexmachina I'm adding my comments here rather than on the PR.

Sorry for the delay in responding to PRs. It is bad of me to leave you in limbo.

On why I haven't merged: I'm on the fence. I'm not a fan of dropping "details" information (as opposed to trimming the time field which has very redundant data), of special casing output for some of the levels, and of the potential backward incompat surprise to those that use 'short' output frequently. The original intention of the 'short' output mode was to try to recover some _width_ to attempt to fit in 80 columns (http://trentm.com/node-bunyan/bunyan.1.html#OUTPUT-FORMATS). I work with some 80-columns nerds. :) IOW, less output, but not less information.

I'm wondering if what you are describing would fit better in a new output mode name. Given that "short" intends short on the horizontal, perhaps "thin" for this? Having short, thin and long starts to be not so self-explanatory, so a better name here is definitely welcome.

How about quiet?

@trentm, I'd like to reopen this issue. How about --ouput quiet to remove the "details" from CLI output except when level is error, so that error info is still displayed. If you're okay with this I'll update my PR and resubmit.

@aexmachina good name, +1 to a PR adding this. And thanks for your patience.

Eventually I really want a way to extend this with separate code (i.e. one can add an output format without having to get something PR'd into bunyan itself). But I'm not going to have time for that for a long while.

:+1:

Hey is there an update on getting this merged?

I'd love to see this merged as well!

Was this page helpful?
0 / 5 - 0 ratings