search

quchen / prettyprinter

A modern, extensible and well-documented prettyprinter.
BSD 2-Clause "Simplified" License
293 stars 34 forks source link

Incorrect usage of MINIMAL pragma? #218

Closed dwincort closed 2 years ago

dwincort commented 2 years ago

I'm not sure why there's a MINIMAL pragma for the Pretty class considering that there's a default implementation for pretty. This seems a lot like a similar issue with toJSON in aeson, and the solution looks like the pragma shouldn't be there.

sjakobi commented 2 years ago

Yes, let's remove the pragma.