rethink signature of SimpleInterface.info() and make fields optional first argument

[tripodsan]

Is your feature request related to a problem? Please describe.

the current signature of the SimpleInterface logging methods are:

info(string: message)
infoFields(string: message, object: fields)

It would be more aligned with the bunyan logger, and would allow for potential string interpolation, if the signature would be:

info(string: message)
info(object: fields, string:message) // overload

Describe the solution you'd like
create a breaking change and:

• remove the SimpleInterface.*Fields() methods
• change the signature so that the first argument is either a fields object or a string message.

[koraa]

the current signature of the SimpleInterface logging methods are:

info(string: message)
infoFields(string: message, object: fields)

Thatis not quite correct; the signature is:

info(...any)
infoFields(...any, object: fields)

In this way they are very similar to console.log; an overload as you are suggesting would be ambiguous:

info(...any)
info(...any, object: fields) // could also match the first overload

I agree that the Fields methods are not pretty, but they are unambiguous…
Although I do agree finding a better interface would be great, so if we can find a better interface that preserves the console.log like functionality while staying unambiguous…that would be great!