Skip to content

[#825] Remove 'OpSize' modules

Roman Melnikov requested to merge rvem/#825-remove-opsize-machinery into master

Description

Problem: Functionality provided by 'OpSize' modules doesn't seem to match the reference implementation anymore. Additionally, seems like this code isn't actually used anywhere.

Solution: It doesn't make sense to support this code in the future since the only place where it can be potentially used is fee calculation in 'morley-client', which currently uses node RPC responses to estimate operations sizes.

Related issue(s)

Resolves #825 (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 Roman Melnikov

Merge request reports