Version 2

[JeanMertz]

• no more global logger (now using logger.New which returns a new Zap logger)
• better Stackdriver integration (thanks to https://github.com/tommy351/zap-stackdriver, I'll upstream the changes I made when I get the time)
• Glide -> Dep
• proper tests
• drop tier log field
• drop environment log field
• add easy to use test logger at logger.TestNew
• add CI integration

note: you can still easily set the tier and environment fields yourself on initialisation:

logger := logger.Must(logger.New(
  "my-service",
  "my-version",
  zap.Fields(
    zap.String("tier", "my-tier"),
    zap.String("environment", "my env"),
  ),
))

For more details on the Stackdriver log format, see:

• https://cloud.google.com/logging/docs/reference/v2/rest/v2/LogEntry
• https://cloud.google.com/error-reporting/docs/formatting-error-messages

[jurre]

I'd expect NewTest, if all the "constructor" functions begin with New it helps with autocompletion etc

[JeanMertz]

I prefer TestNew, as this is supposed to a "public testing API" as described by Mitchell Hashimoto.

See:

• https://speakerdeck.com/mitchellh/advanced-testing-with-go?slide=52
• https://www.youtube.com/watch?v=yszygk1cpEc

Right now, it's only a single function, but there could be more down the line, and then it makes sense to always prefix them with Test (and move them into testing.go, but I didn't see the point of that for this one function, but maybe I should).

Also, in terms of autocompletion, all editors I use actually use fuzzy matching, so not sure if that's a reason to change this default.