haskell-suite / haskell-src-exts

Manipulating Haskell source: abstract syntax, lexer, parser, and pretty-printer
Other
193 stars 94 forks source link

Support QuantifiedConstraints #427

Closed AshleyYakeley closed 4 years ago

AshleyYakeley commented 5 years ago

I've simplified the Asst and PAsst types so that a context assertion is either a type, an implicit parameter, or a parenthesised assertion.

ttuegel commented 5 years ago

Lack of support for QuantifiedConstraints is breaking tool support (such as stylish-haskell) for GHC 8.6. Is there any way I could help get this merged?

mpickering commented 5 years ago

I didn't see this PR. Looks reasonable to me but CI is failing.

AshleyYakeley commented 5 years ago

Looks like some warnings on older GHCs, shouldn't be too hard.

AshleyYakeley commented 5 years ago

Before I fix it, is it intended that haskell-src-exts support GHC versions as far back as 7.6? Note that 7.10 (which this PR works with) was released almost four years ago.

mpickering commented 5 years ago

I suppose 7.6 is quite old now but seeing as it wasn't much effort to keep supporting I didn't see any reason to remove it. It's probably fine to remove releases before 7.10.

AshleyYakeley commented 5 years ago

OK, so that fixed GHC 7.8. On GHC 7.6, two tests cause stack overflow.

konn commented 5 years ago

Any update on this?

awpr commented 5 years ago

Associating this with related issues #433 and #419.

cpeikert commented 4 years ago

This would be a very useful feature for me. Is there update on whether it can be merged?

DanBurton commented 4 years ago

It can! It's been on my backlog for a while. I had a mergeable version of this (see #441) about a month ago. I went ahead and merged & released as haskell-src-exts-1.22.0.

I made a slight mistake with the release in that the full changelog updates for this release were not properly published. See the in-repo changelog for more complete details.