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

re-adding file watching #2629

Closed
wants to merge 3 commits into from
Closed

Conversation

axdank
Copy link
Contributor

@axdank axdank commented May 31, 2022

Re-added the files so they can be merged (Taken from PR #588)

If it can be merged so that someone more knowledgeable in helix code can implement missing functions such as the file lookup required in (LSP - DidChangeWatchedFiles Notification).

One of the problems to be solved: #2479

@kyrime
Copy link
Contributor

kyrime commented Jun 1, 2022

Hi! I'm working on #2479. Basic functionality is working, should be ready to open a PR for it once I find time to work on it a bit more. Watcher implementation is not quite the same as #588 so this PR would make it a bit awkward with a conflicting implementation.

@axdank
Copy link
Contributor Author

axdank commented Jun 1, 2022

@kyrime Ohhh that good, I based on this code and I managed to solve the problem that commented in #2479, I added the ( DidChangeWatchedFiles Notification ) for this case, and it worked. But much better if you are already working on it, I will proceed to close this PR to not cause conflicts with yours.

Here is a video of my implementation:
2022-06-01 18-40-18

@axdank axdank closed this Jun 1, 2022
@axdank axdank deleted the add_file_watcher branch July 21, 2022 14:22
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