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

Make server side pprint the default pretty printer #1650

Closed
PEZ opened this issue Apr 1, 2022 · 0 comments
Closed

Make server side pprint the default pretty printer #1650

PEZ opened this issue Apr 1, 2022 · 0 comments

Comments

@PEZ
Copy link
Collaborator

PEZ commented Apr 1, 2022

In #1621 we made client side calva the default pretty printer. That has many benefits, but also an important drawback: Evaluating infinite sequences never returns a result. Because the length limit is applied on the results (which will never happen).

This can be a bit puzzling when it happens, especially for beginners with Clojure. Even myself, who have some Clojure under my belt now, wondering for a few seconds what was going on when first attempting to record this video:

Run.to.the.Hills-Clojure.mp4

Credit for the code goes to @orestis and other fine folks over at Clojurians. See this tweet:

https://twitter.com/orestis/status/1509553038268243976

@PEZ PEZ mentioned this issue Apr 1, 2022
16 tasks
@bpringe bpringe closed this as completed in 18020dc Apr 1, 2022
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

1 participant