-
Notifications
You must be signed in to change notification settings - Fork 805
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
megaparsec 6.0 #2666
Comments
|
Entirely redundant with the above, but do also note: hspec-megaparsec-1.0.0 (Mark Karpov markkarpov92@gmail.com @mrkkrp) is out of bounds for:
|
My parser seems to break when doing a naive move to v6. It will take me a while to find out why. |
Some porting notes, perhaps better here than on reddit: added where needed:
replaced Dec with MPErr and added:
added these as Text.Megaparsec.String/Text.Megaparsec.Text seem to have gone away:
|
The library itself recommends setting the error type (where |
Here's a Text.Megaparsec.Compat module, useful for supporting megaparsec <6 and >=6. |
@bartavelle If you have any trouble switching please open an issue, I'll help. |
@fosskers: this was unfortunate, but I removed "versions" from the build plan since it could not satisfy both the new version of base (which we are upgrading to now) and the old version of Megaparsec (which we are upgrading later). |
@mrkkrp I'll try to pinpoint the problem next week. The only meaningful change I did was changing an |
@DanBurton but I bumped my Hmm... actually I don't think I can support any megaparsecs less than 6 now, since a user requested that I expose some of the internal parsers that |
Updated:
|
@mrkkrp Turned out my parser had bugs, but they did not surface with megaparsec 5! |
released language-puppet-1.3.9 |
Fixed for |
|
Can we perhaps update the checkboxes in the first post? |
Checkboxes updated |
@mihaimaruseac |
I see they are updated already. |
Yeah. I see they are already checked. Thanks! @mihaimaruseac |
I would love to see megaparsec updated to |
You are right! and I am sorry for being the laggard here. I updated the hledger packages early but have been hung up on the release process. I will release them today. |
@simonmichael totally cool: will get them right back as soon as the releases have been made. Thanks! |
Thanks @cdornan. |
Megaparsec-6 has been blocked for 4 weeks so we have decided to (temporarily we hope) evict the remaining blockers and restore them as soon as they are compatible with megaparsec-6. I have raised issue in the three projects concerned:
|
Now that Megaparsec 6 is in nightly builds, can we allow newest |
These packages now support megaparsec 6 and should be free of other dep issues.
These packages now support megaparsec 6 and should be free of other dep issues.
megaparsec-6.0.0 (Mark Karpov markkarpov92@gmail.com @mrkkrp) is out of bounds for:
The text was updated successfully, but these errors were encountered: