search

circuithub / rel8

Hey! Hey! Can u rel8?
https://rel8.readthedocs.io
Other
150 stars 38 forks source link

Allow hedgehog 1.2 in test suite #248

Closed sternenseemann closed 9 months ago

sternenseemann commented 1 year ago

That version is now shipped with Stackage LTS 21. The test suite builds and passes for me with said hedgehog version, so a hackage revision would be appreciated!

ocharles commented 1 year ago

I'm happy to bump this, but why do you need a Hackage revision? Rel8 only uses Hedgehog for its test suite, so this version shouldn't matter for normal development. Does Stack run tests or something?

sternenseemann commented 1 year ago

stack does not (Stackage would, but that's not relevant here I believe), but nixpkgs does. A revision is not strictly necessary, but it's usually helpful to be able to clean up constraint-related workarounds.

ocharles commented 1 year ago

Ah, yes, nixpkgs runs tests. Cool, well either way - this should be done! Thanks for the issue.

shane-circuithub commented 9 months ago

This was fixed in #278.