Skip to content

[#857] Clean up SingI1

David Feuer requested to merge dfeuer/#857-singi1-simplification into master

Description

We don't need multiple SingI1 instances now that QuantifiedConstraints is available, so let's not have them.

Related issue(s)

Resolves #857 (closed) by giving SingI1 a quantified superclass constraint and a single polymorphic instance, and turning its method into a function.

Changes the name of SingI1 to SingIOne to resolve a potential name clash with the similar (but inferior) class in singletons.

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 David Feuer

Merge request reports