Pino: remove the `silent` logger method

[davidmarkclements]

We support silent as .a special public logger method, and test it but it's not in the docs

[mcollina]

should we just remove it in the next semver-major?

[jsumners]

No. It’s really useful for tests.

[davidmarkclements]

hey @jsumners can you elaborate on how it's useful for tests? I don't see the usefulness, and currently think we should remove but if it's significantly useful and there's no equally trivial alternative approach in testing then we should probably keep it

[jsumners]

If you are testing a something that uses Pino for logging, and you don't want to see those logs all over your tests, it's helpful to be able to silence them.

[davidmarkclements]

Absolutely.. I think there's some confusion here. There's a method called logger.silent, its a no-op and there's no use case (that I can think of) for it. setting the level to silent, is different thing and the internal implementation doesn't rely on the existence of a silent method

[jsumners]

Cool.

[davidmarkclements]

Perfetto - let's remove