Skip to content

[#237] Include `KnownValue` into `IsExpr`

Moremi Vannak requested to merge rinn7e/#237-include-knownvalue-in-isexpr into master

Description

Problem: There seems to be no case of IsExpr ex a where KnownValue a constitutes a problem, however we are required add the latter every time we use the former (where a is generic).

Solution: Include KnownValue a into IsExpr ex a and remove unnecessary use of KnownValue.

Related issue(s)

Resolves #237 (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)

Merge request reports