Skip to content

prettyprinter-compat-ansi-wl-pprint is not quite a drop-in replacement as claimed #246

Description

@georgefst

This was noticed in ekmett/ansi-wl-pprint#32 (comment), but I've found some other incompatibilities:

  • There's no Pretty a => Pretty (Maybe a) instance, which old code may rely on.
  • We have Doc = Doc AnsiStyle, which means that any existing Pretty instances which use basic functions like text no longer compile, since prettyprinter's pretty has to be annotation-polymorphic.

These are not easily solved, so I think the README ought to just be clearer about the limitations.

This has caused issues for some ansi-wl-pprint-dependent packages, such as tmp-postgres.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions