Skip to content
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

Compact mode destroys laziness #113

Open
georgefst opened this issue Jun 13, 2022 · 3 comments
Open

Compact mode destroys laziness #113

georgefst opened this issue Jun 13, 2022 · 3 comments

Comments

@georgefst
Copy link
Collaborator

Oh, and I realized that currently pretty-simple doesn't parse and pretty-print lazily with the compact option:

> pPrintOpt CheckColorTty defaultOutputOptionsDarkBg { outputOptionsCompact = True }  (repeat 3)

This doesn't print anything. Maybe I should make a separate issue about this...

Originally posted by @cdepillabout in #84 (comment)

@georgefst
Copy link
Collaborator Author

This is probably my fault. I'd never even thought to test it.

@cdepillabout
Copy link
Owner

Yeah, I think the real problem here is that pretty-simple doesn't have any tests checking the whole laziness thing!

@georgefst
Copy link
Collaborator Author

Yeah, I think the real problem here is that pretty-simple doesn't have any tests checking the whole laziness thing!

We have at least one! I remember from #67 (comment).

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

No branches or pull requests

2 participants