-
Notifications
You must be signed in to change notification settings - Fork 9
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
Use prettyprinter-compat-ansi-wl-pprint as an implementation #32
Conversation
@RyanGlScott, do you have an opinion on this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great!
@@ -1,7 +1,6 @@ | |||
cabal-version: >= 1.10 | |||
name: ansi-wl-pprint | |||
version: 0.6.9 | |||
x-revision: 4 | |||
version: 1.0.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why pick version 1.0.2
instead of, say, 1.0.0
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's the same version as https://hackage.haskell.org/package/prettyprinter-compat-ansi-wl-pprint
315bab5
to
16e620b
Compare
Note that there are other incompatibilities: quchen/prettyprinter#246. These ideally ought to have been mentioned in the release notes, but at least this package was given a major version bump when switching over. |
Resolves #26, as with this patch you'll end up using
prettyprinter
.Before:
After:
The green text is done with
and that doesn't work with
Prettyprinter
'sDoc
(which doesn't use annotations inShow
instance).Otherwise compat shim seems to be quite complete, as none of imports needed to be removed.
This will finally make
ansi-wl-pprint
obsolete.