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

Rebind *print-namespace-maps* #807

Open
vemv opened this issue Aug 26, 2023 · 3 comments
Open

Rebind *print-namespace-maps* #807

vemv opened this issue Aug 26, 2023 · 3 comments

Comments

@vemv
Copy link
Member

vemv commented Aug 26, 2023

IME, many people dislike the *print-namespace-maps* default behavior, i.e. print namespaced maps at the repl.

It seems to me that cider-nrepl could change this default?

Right now, we observe options such as:

  nrepl.middleware.print/options     (dict
                                       right-margin 70)

If we change the default, these options look like a nice place to let users opt out.

@bbatsov
Copy link
Member

bbatsov commented Aug 26, 2023

Can you elaborate on what's the issue here? Isn't CIDER using the same default as Clojure?

@vemv
Copy link
Member Author

vemv commented Aug 26, 2023

Yes, it's the same default (true) that people dislike.

Maybe we can simply add that option here. Then, CIDER side, one could customize cider-print-options (and we could look into changing the default later).

@bbatsov
Copy link
Member

bbatsov commented Aug 28, 2023

Making this configurable sounds reasonable.

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