Linter fixes and cleanup

Author: Paweł Rozynek

emf/logger.go

@@ -41,11 +41,11 @@ func WithTimestamp(t time.Time) LoggerOption {
 	}
 }
 
-// NewWith creates logger with reasonable defaults for Lambda functions:
+// New creates logger with reasonable defaults for Lambda functions:
 // - Prints to os.Stdout.
 // - Context based on Lambda environment variables.
 // - Timestamp set to the time when NewWith was called.
-// Specify NewOptions to customize the logger.
+// Specify LoggerOptions to customize the logger.
 func New(opts ...LoggerOption) *Logger {
 	values := make(map[string]interface{})
 

emf/logger_internal_test.go

@@ -46,7 +46,6 @@ func TestNew(t *testing.T) {
 
 // loggersEqual returns a non-nil error if the loggers do not match.
 // Currently it only checks that the loggers' output writer and timestamp match.
-// TODO: expand the checks here as more NewOptions are added.
 func loggersEqual(actual, expected *Logger) error {
 	if actual.out != expected.out {
 		return fmt.Errorf("output does not match")