Skip to content

starting to bring to new version of ghc#30

Open
andrewufrank wants to merge 2 commits intoarademaker:masterfrom
andrewufrank:changesfor13.12
Open

starting to bring to new version of ghc#30
andrewufrank wants to merge 2 commits intoarademaker:masterfrom
andrewufrank:changesfor13.12

Conversation

@andrewufrank
Copy link
Copy Markdown

I tried to bring the package to a more current ghc (8.6.4) with lts-13.12 but found a number of errors, mostly with regards to Megaparsec and error handling - which I do not understand very well.
Could you try to fix them? Thank you!

@andrewufrank
Copy link
Copy Markdown
Author

I fixed some trivial errors (using the instruction of the changelog of megaparsec 7.0 as instructions) but I could not fix the last error.
probably you should check that the new version is compatible with 8.4 and 8.2 of ghc.
Thank you for a useful tool!

@odanoburu
Copy link
Copy Markdown
Collaborator

hi @andrewufrank, thanks for this!

I've incorporated your changes into a new PR and updated a few small things to get megaparsec to work again. can you test and see if there's any change in behaviour?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants