fix: default to display stack trace

oclif · Feb 4, 2018 · 20d719c · 20d719c
1 parent 67d28b2
commit 20d719c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/src/config.ts b/src/config.ts
@@ -32,6 +32,7 @@ export class Config {
   errorsHandled = false
   context: any = {}
   errlog?: string
+  showStackTrace = true
 
   constructor() {
     this.debug = process.env.DEBUG === '*'

diff --git a/src/errors.ts b/src/errors.ts
@@ -71,7 +71,7 @@ export function getErrorMessage(err: any, opts: {stack?: boolean} = {}): string
     message,
   ].join('')
 
-  if (opts.stack || process.env.CI || severity === 'fatal' || config.debug) {
+  if (config.showStackTrace || opts.stack || process.env.CI || severity === 'fatal' || config.debug) {
     // show stack trace
     let stack = err.stack || inspect(err)
     stack = clean(stack, {pretty: true})

diff --git a/test/errors.test.ts b/test/errors.test.ts
@@ -7,6 +7,7 @@ describe('errors', () => {
   .env({CI: null})
   .stderr()
   .it('warns', async output => {
+    cli.config.showStackTrace = false
     cli.warn('foobar')
     if (process.platform === 'win32') {
       expect(output.stderr).to.equal(' !    Warning: foobar\n')