Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

starting to bring to new version of ghc #30

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

andrewufrank
Copy link

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
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
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.

None yet

2 participants