Switch to a more advanced pretty printing library

knrafto / language-bash

Parse and pretty-print Bash shell scripts

BSD 3-Clause "New" or "Revised" License

35 stars 9 forks source link

Switch to a more advanced pretty printing library #28

Closed mmhat closed 4 years ago

mmhat commented 4 years ago

It would be nice if we support Text pretty printing in addition to simple Strings. We could use grouping features to improve the output as well. Personal choice: prettyprinter

mmhat commented 4 years ago

Draft i here. Passes all tests in #30. But I don't want to make a pull request before discussing this issue.

knrafto commented 4 years ago

I'm all for it, prettyprinter seems to be the best choice nowadays.

knrafto commented 4 years ago

I should add that the only reason this library doesn't use prettyprinter is that prettyprinter didn't exist when this project started. We should upgrade the other dependencies too (e.g. Parsec -> Megaparsec #31)