Skip to content

prettyprinter-ansi-terminal: add 'text' flag#279

Merged
sjakobi merged 1 commit intohaskell-prettyprinter:masterfrom
Bodigrim:text-flag
Mar 29, 2026
Merged

prettyprinter-ansi-terminal: add 'text' flag#279
sjakobi merged 1 commit intohaskell-prettyprinter:masterfrom
Bodigrim:text-flag

Conversation

@Bodigrim
Copy link
Copy Markdown
Contributor

@Bodigrim Bodigrim commented Mar 5, 2026

As discussed at #264 (comment)

@sjakobi
Copy link
Copy Markdown
Collaborator

sjakobi commented Mar 6, 2026

What do you think, @Ei30metry, @a-02?

@a-02
Copy link
Copy Markdown
Collaborator

a-02 commented Mar 6, 2026

Glad there's a decent description here finally. Nothing I can find objectionable, but I'll wait for @Ei30metry.

Copy link
Copy Markdown
Collaborator

@sjakobi sjakobi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to integrate this into the existing no-text CI.

Is there already any indication that optparse-applicative would use this API, or that there would be another "consumer"?

@Bodigrim
Copy link
Copy Markdown
Contributor Author

I updated no-text CI job to include prettyprinter-ansi-terminal.

optparse-applicative would need a simple patch (replacing Text.unpack . renderLazy with renderString), which I will implement if this get merged and released.

@sjakobi sjakobi requested a review from Ei30metry March 16, 2026 10:21
@Bodigrim
Copy link
Copy Markdown
Contributor Author

Anything else I can do to get this merged please?

@sjakobi sjakobi merged commit d70be67 into haskell-prettyprinter:master Mar 29, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants