Skip to content

Add ability for plugins to handle file change notifications #1585

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

Closed
wants to merge 16 commits into from

Conversation

alanz
Copy link
Collaborator

@alanz alanz commented Mar 17, 2021

This allows alternate parsers to be used, such as tree-sitter.

wz1000 and others added 16 commits February 10, 2021 22:24
some progress

Mostly everything except LanguageServer.hs

make it compile

make it work

fix benchmarks

update

tweaks

fix configuration and tests

simplify handlers

Update to renamed lsp/lsp-types modules

redo plugin api and get library to compile

fill in some missing details

fix main

fix rebase
Like pluginHandlers, but for notifications

At present the last one in the chain wins, so if one is set it
overrides the one built into ghcide
@alanz alanz requested a review from pepeiborra March 17, 2021 14:54
@jneira
Copy link
Member

jneira commented Mar 17, 2021

this needs a heavy rebase on máster, no? maybe we could set it as draft meanwhile?

@alanz alanz marked this pull request as draft March 17, 2021 15:31
@alanz
Copy link
Collaborator Author

alanz commented Mar 17, 2021

@jneira

this needs a heavy rebase on master, no?

Yes, I made it a draft as you suggested, it was more to show @pepeiborra what I have done, and it definitely needs a rebase.

@pepeiborra
Copy link
Collaborator

@alanz I'll rebase and send a new PR later this week

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.

4 participants