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

with-out-str #31

Closed
corasaurus-hex opened this issue Aug 17, 2022 · 2 comments
Closed

with-out-str #31

corasaurus-hex opened this issue Aug 17, 2022 · 2 comments

Comments

@corasaurus-hex
Copy link
Collaborator

Is this something we want to add? If we do want to add this then we should probably do this before we do pr/prn, right?

@borkdude
Copy link
Member

prn and pr-str are already there: they use JSON.stringify.

@borkdude
Copy link
Member

I consider prn in squint totally broken since it relies in JSON.stringify which ignores a lot of stuff when it's not representable as JSON. I might want to switch to another implementation later which prints EDN-like stuff which can be copy-pasted back into the code. This will be a breaking change. I'll revisit with-out-str after that.

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