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

How to make lsp-haskell more loosely tied to haskell-mode? #181

Open
magthe opened this issue Oct 15, 2023 · 4 comments
Open

How to make lsp-haskell more loosely tied to haskell-mode? #181

magthe opened this issue Oct 15, 2023 · 4 comments

Comments

@magthe
Copy link

magthe commented Oct 15, 2023

In my experimenting with making a Haskell mode that uses tree-sitter I ended up having to make some changes to lsp-haskell to detach it from haskell-mode. My very hacky change is here: magthe@2084ff1

I'd love some input on how to do this more properly and turn it into a PR eventually.

@michaelpj
Copy link
Collaborator

No idea tbh. Probably a question for @yyoncho ?

I'm not sure what's wrong with just adding haskell-ng-mode to the set of modes that we are enabled for? i.e. just do what is already done for e.g. haskell-literate-mode?

@magthe
Copy link
Author

magthe commented Nov 7, 2023

Well, I think it's mostly a question of cleanliness, if lsp-haskell depends on both packages, then both will be installed all the time. Even if it only makes sense to use one or the other, never both.

@michaelpj
Copy link
Collaborator

Your patch doesn't make lsp-haskell depend on haskell-ng-mode? Or am I missing something? do we need a dependency? or is that just an elisp symbol for the mode name?

@magthe
Copy link
Author

magthe commented Nov 7, 2023

No, it only removes completely the dependency on haskell-mode so I can avoid having it around while working on haskell-ng-mode.

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

No branches or pull requests

2 participants