Skip to content

fix: disable workspace/didChangedWatchedFiles dynamyc registration #352

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

Conversation

GuillaumeLagrange
Copy link
Contributor

@GuillaumeLagrange GuillaumeLagrange commented Apr 12, 2024

This causes freezes on large repos

Dirty and temporary fix for #312
Do not merge yet

Copy link
Contributor

Review Checklist

Does this PR follow the Contribution Guidelines? Following is a partial checklist:

Proper conventional commit scoping:

  • For example, fix(lsp): some lsp-related bugfix

  • Pull request title has the appropriate conventional commit prefix.

If applicable:

  • Tested
    • Tests have been added.
    • Tested manually (Steps to reproduce in PR description).
  • Updated documentation.
  • Updated CHANGELOG.md

@mrcjkb
Copy link
Owner

mrcjkb commented Apr 12, 2024

Hey 👋

I don't think we should disable this feature for all users, given there's a workaround.
It probably needs a fix in neovim itself, as it will likely have the same issue with other language servers.

@GuillaumeLagrange
Copy link
Contributor Author

Hey, of course this draft PR was more to illustrate the issue comment #312 (comment)

For some reason, the fix does not work on my end, maybe I messed up something. Feel free to close this PR. I do agree that this is not a real fix as it is.

@mrcjkb mrcjkb closed this Apr 12, 2024
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