- Notifications
You must be signed in to change notification settings - Fork18
Support nested indentation rules#70
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Support nested indentation rules#70
Uh oh!
There was an error while loading.Please reload this page.
Conversation
44af05e toa0aca89CompareUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
bbatsov commentedApr 4, 2025
Yeah, that's fine. Perhaps we can eventually teach CIDER (and maybe Great change overall! |
a0aca89 toec50b30Comparec8286e2 intoclojure-emacs:mainUh oh!
There was an error while loading.Please reload this page.
Hopefully the final piece that concerns semantic indentation.
This change generalize the way we detect indentation for macros and special forms and adds support for nested indentation rules (so, for example, separate function
clojure-ts--match-method-bodyis not needed anymore).I changed to format of
clojure-ts-semantic-indent-rules(hopefully it's OK for the version that has not been yet officially released) from(:block n)to((:block n))or optionally((:block n) (:inner n)), which is more consistent with cljfmt format.New format allows defining rules like
("letfn" . ((:block 1) (:inner 2 0)))).clojure-ts--dynamic-indent-for-symbolnow converts not only top level rule, but the entire spec.Before submitting a PR mark the checkboxes for the items you've done (if you
think a checkbox does not apply, then leave it unchecked):
M-x checkdocand fixed any warnings in the code you've written.Thanks!