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

feat: inherit Eglot's nix-mode settings #38

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

Icy-Thought
Copy link
Contributor

Reason for this change is to inherit the defined configuration for nix-mode within eglot-server-programs to automate the process of setting up Eglot for this mode.

Current (as of commit date) value for nix-mode found within eglot-server-programs:

(nix-mode . ,(eglot-alternatives '("nil" "rnix-lsp" "nixd")))

@Icy-Thought Icy-Thought requested a review from a team as a code owner July 10, 2024 06:22
Copy link
Member

@purcell purcell left a comment

Choose a reason for hiding this comment

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

I guess derived-mode-add-parents is new in Emacs HEAD: it's not in any released version of Emacs. Snippets like this do appear to be routinely used for this purpose in Emacs, so I'm generally in favour.

@Icy-Thought
Copy link
Contributor Author

That's the main reason why I encapsulated the function with a functionp statement! Not everyone will have access to this new feature, but if they do then they get the extra benefit that it could bring to this package! :)

@purcell
Copy link
Member

purcell commented Jul 10, 2024

That's the main reason why I encapsulated the function with a functionp statement!

Yep, appreciated that, thanks. :)

@purcell
Copy link
Member

purcell commented Jul 10, 2024

Comment was meant as "huh, today I learned" as much as anything.

@Icy-Thought
Copy link
Contributor Author

Icy-Thought commented Jul 10, 2024

Aha, gotcha! Glad to help out! 😄

@purcell purcell merged commit 59ad176 into nix-community:trunk Jul 10, 2024
16 checks passed
@purcell
Copy link
Member

purcell commented Jul 10, 2024

Merged now, thank you!

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