Skip to content

IoHost used as only logger everywhere #32346

@mrgrain

Description

@mrgrain

With #32345 completed, we can move to using the IoHost.

  • create a global singleton CliIoHost
  • change every log statement to use the new CliIoHost
  • remove all other logger implementations, instead every call must call ioHost.notify(...)
  • We cannot have helpers to call notify
  • However we can have helper methods to create IoMessages of the correct type, i.e. ioHost.notify(error("momo made a boo boo")) is fine

This should not require any changes to tests. If so, explain why.

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions