VR 405 - logging

[rmlearney-digicatapult]

Pull Request

Checklist

• Have you read Digital Catapult's Code of Conduct?
• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• I have added tests that prove my fix is effective or that my feature works.
• New and existing unit tests pass locally with my changes.

PR Type

Please delete options that are irrelevant.

• Bug Fix

Linked tickets

https://digicatapult.atlassian.net/browse/VR-405

High level description

Fix empty log messages, removed duplicate logging, swapped to pino-pretty for local dev

Detailed description

Use string interpolation in internal error handlers Error('${err}'), removed duplicate log entries where we were passing a cli log.warn('message') and a throw new Error('message') at the same time

Switched to pino-pretty for cli dev

Describe alternatives you've considered

Operational impact

Additional context