Skip to content

[#936] Add an Ord instance for AddressWithAlias

Description

Problem: while there's no single well-defined ordering for AddressWithAlias, there is an ordering, and it's useful to have an Ord instance on it, so that it could be used e.g. as a Map key.

Solution: add the instance. Also explain the motivation for this type's existence and how its Ord instance is arbitrary.

Related issue(s)

Resolves #936 (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 Nikolay Yakimov

Merge request reports