Skip to content

Add support for automatic aligning forms #85

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

Merged

Conversation

rrudakov
Copy link
Contributor

Close #16

I copied all related tests from clojure-mode with one exception. should handle incomplete sexps won't work, because syntax tree will be broken.

I also found a few issues in the clojure-ts-align after running full clojure-mode test suite. All the issues are fixed now.


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):

  • The commits are consistent with our contribution guidelines.
  • You've added tests (if possible) to cover your change(s). Bugfix, indentation, and font-lock tests are extremely important!
  • You've run M-x checkdoc and fixed any warnings in the code you've written.
  • You've updated the changelog (if adding/changing user-visible functionality).
  • You've updated the readme (if adding/changing user-visible functionality).

Thanks!

@rrudakov rrudakov force-pushed the feature/16-align-forms-automatically branch from e4bba07 to cb6f016 Compare April 23, 2025 18:47
@rrudakov
Copy link
Contributor Author

Ready for review.

@rrudakov rrudakov force-pushed the feature/16-align-forms-automatically branch 3 times, most recently from 4b089f7 to fd9e791 Compare April 23, 2025 19:45
@rrudakov rrudakov force-pushed the feature/16-align-forms-automatically branch from fd9e791 to 9b7ac71 Compare April 25, 2025 14:07
@bbatsov bbatsov merged commit e960a90 into clojure-emacs:main Apr 25, 2025
3 checks passed
@bbatsov
Copy link
Member

bbatsov commented Apr 25, 2025

Looks solid to me. Thanks!

@rrudakov rrudakov deleted the feature/16-align-forms-automatically branch April 25, 2025 15:15
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

Successfully merging this pull request may close these issues.

Add suport for clojure-align-forms-automatically
2 participants