Skip to content

[#426] Keep MorleyLogs in WriterT in interpreter

Konstantin Ivanov requested to merge martoon/#426-morley-logs-in-writer into master

Description

Currently MorleyLogs are kept in StateT, and this design has some flaws.

This MR moves MorleyLogs to a dedicated WriterT layer.

Probably the same thing should be done for Integrational framework (and so for Cleveland), I'm leaving this for #461 (closed). If such change is not necessary for that issue, then we can live without it in our testing frameworks for now.

Related issue(s)

Resolves #426 (closed)

Checklist for your Merge Request

Related changes (conditional)

  • Tests (see short guidelines)

    • If I added new functionality, I added tests covering it.
    • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • Documentation

    • I checked whether I should update the docs and did so if necessary:
    • I updated changelog files of all affected packages released to Hackage if my changes are externally visible.

Stylistic guide (mandatory)

Edited by Konstantin Ivanov

Merge request reports