Skip to content

Fix some issues with short anonymous functions #86

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
merged 1 commit into from
Apr 28, 2025

Conversation

rrudakov
Copy link
Contributor

@rrudakov rrudakov commented Apr 25, 2025

I noticed that some functionality didn't work for short anonymous functions, such as syntax highlighting of built-ins, indentation rules and vertical aligning of forms. Mostly because they are not list_lit nodes, but separate node type.

I also discovered and reported another bug in Emacs and introduced a temporary workaround.

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/improve-short-lambdas branch 2 times, most recently from 6ffbd9f to 0e4252d Compare April 26, 2025 07:03
@rrudakov rrudakov force-pushed the feature/improve-short-lambdas branch from 0e4252d to 805f31e Compare April 28, 2025 07:13
@rrudakov rrudakov requested a review from bbatsov April 28, 2025 07:23
@bbatsov bbatsov merged commit 43dbadd into clojure-emacs:main Apr 28, 2025
3 checks passed
@rrudakov rrudakov deleted the feature/improve-short-lambdas branch April 28, 2025 08:37
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.

2 participants