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

Inconsistent indent in multi-line anonymous fn when zprint style is indent-only #243

Closed
tflavin opened this issue Jun 10, 2022 · 3 comments

Comments

@tflavin
Copy link

tflavin commented Jun 10, 2022

As written:

#(let [really-long-text-so-it-puts-on-two-lines %]
         really-long-text-so-it-puts-on-two-lines)

Classic:

#(let [really-long-text-so-it-puts-on-two-lines %]
   really-long-text-so-it-puts-on-two-lines)

Indent only:

#(let [really-long-text-so-it-puts-on-two-lines %]
  really-long-text-so-it-puts-on-two-lines)
@kkinnear
Copy link
Owner

Thanks for noticing this! I have also fixed this, and it will be in the next release of zprint (almost certainly 1.2.4). I appreciate your attention to detail applied to :indent-only! Thanks again!

@tflavin
Copy link
Author

tflavin commented Jun 21, 2022

No problem - I first wanted to evaluate it across our codebase and initially thought it was just a difference in opinion from our current styling but then realized that it was inconsistent with classic. I think after 1.2.4 we'll be able to use :indent-only

@kkinnear
Copy link
Owner

kkinnear commented Aug 2, 2022

Fixed in 1.2.4, just out.

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