Skip to content

Use newer vinyl and remove one NFData instance

Ivan Gromakovskii requested to merge gromak/newer-vinyl into master

Description

Problem: we have NFData instance for Rec defined in our code. Recently it was merged into vinyl package, so we can freely remove some code.

Solution:

  1. Update Stackage resolver to the closest one with vinyl-0.13.1 where this instance was added.
  2. Remove this instance.
  3. Require vinyl-0.13.1 to avoid breaking something. This requirement is transitively imposed on packages depending on morley.

Related issue(s)

None, I just need it for another project.

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