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

Update apheleia-indent-lisp-buffer to respect local variables #286

Merged
merged 2 commits into from
Mar 2, 2024

Conversation

tpeacock19
Copy link
Contributor

I ran into some issues using apheleia while contributing to a project
that works with indent-tabs-mode disabled. This led to this slight
modification of apheleia-indent-lisp-buffer

This allows for any local values for indent-line-function,
lisp-indent-function to be carried over when
formatting. Additionally, added indent-tabs-mode in order to
prevent adding tabs when unwanted.

If there is any additional documentation you would like me to provide,
let me know and I can update.

This allows for any local values for `indent-line-function`,
`lisp-indent-function` to be carried over when
formatting. Additionally, added `indent-tabs-mode` in order to
prevent adding tabs when unwanted.
@raxod502
Copy link
Member

raxod502 commented Mar 2, 2024

Yeah, that makes sense. We should be enabling the major mode before setting the local variables. I think this just needs a changelog entry, no need to update the README as the new behavior is what a user would intuitively expect to happen.

@tpeacock19
Copy link
Contributor Author

I put it under bugs fixed if that works

Copy link
Member

@raxod502 raxod502 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✨ Thanks!

@raxod502 raxod502 merged commit 2fec569 into radian-software:main Mar 2, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants