Merge pull request #49 from JasonEtco/omit-empty-brackets

Omit empty brackets if log.time is falsey

index.js

@@ -120,15 +120,18 @@ module.exports = function prettyFactory (options) {
       log.time = formatTime(log.time, opts.translateTime)
     }
 
-    var line = `[${log.time}]`
+    var line = log.time ? `[${log.time}]` : ''
 
     const coloredLevel = levels.hasOwnProperty(log.level)
       ? color[log.level](levels[log.level])
       : color.default(levels.default)
     if (opts.levelFirst) {
       line = `${coloredLevel} ${line}`
     } else {
-      line = `${line} ${coloredLevel}`
+      // If the line is not empty (timestamps are enabled) output it
+      // with a space after it - otherwise output the empty string
+      const lineOrEmpty = line && line + ' '
+      line = `${lineOrEmpty}${coloredLevel}`
     }
 
     if (log.name || log.pid || log.hostname) {

test/basic.test.js

@@ -276,7 +276,7 @@ test('basic prettifier tests', (t) => {
     const log = pino({ timestamp: null }, new Writable({
       write (chunk, enc, cb) {
         const formatted = pretty(chunk.toString())
-        t.is(formatted, `[undefined] INFO  (${pid} on ${hostname}): hello world\n`)
+        t.is(formatted, `INFO  (${pid} on ${hostname}): hello world\n`)
         cb()
       }
     }))