Skip to content

[#193] Remove tezos-bake-monitor-lib dependency

Moremi Vannak requested to merge rinn7e/#193-tezos-bake-monitor-lib into master

Description

Problem: We use tezos-bake-monitor-lib to implement conversion between Value to JSON by binary round trip. Since we already have Value to binary, tezos-bake-monitor-lib allow us to convert binary to Expression then to JSON.

However, this dependency has a few problem include: not quite stable and not on Hackage which make morley not being able to built on Hackage.

Solution: Remove tezos-bake-monitor-lib and implement the binary round trip ourselves.

Related issue(s)

Resolves #193 (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 Moremi Vannak

Merge request reports