conjure-cp / conjure

Conjure: The Automated Constraint Modelling Tool
Other
96 stars 20 forks source link

Parser update #546

Closed Maurice-Byrne closed 1 year ago

Maurice-Byrne commented 1 year ago

The big Conjure parser overhaul. Not quite ready for merging yet but tentatively putting it up for review. The LSP function is under a separate PR.

Changes:

Known problems:

Testing:

Related issues: Fixes #521 Fixes #554

ozgurakgun commented 1 year ago

Not the most important thing, but if you change line 6 in the Makefile to GHC-9.2 the build in the CI might progress a bit further.

Maurice-Byrne commented 1 year ago

Not the most important thing, but if you change line 6 in the Makefile to GHC-9.2 the build in the CI might progress a bit further.

Looks like the env var is explicitly set in the CI

ozgurakgun commented 1 year ago

Oops, yes. Feel free to change it there if you like.