dotnet / dotnet/roslyn

New file watcher implementation removes watchers before consolidated watcher is created

Open
#83,435 0 comments 0 reactions 0 assignees View on GitHub
Area-IDE
Dominant language
C#
Stars
20.7k
Forks
4.3k
PR merge metrics
PR metrics pending

Description

There is a small problem here that by creating a new file watcher after we removed the other ones, the process of consolidation makes a temporary window where we may not get a notification at all.

_Originally posted by @jasonmalinowski in https://github.com/dotnet/roslyn/pull/83101#discussion_r3140999762_

Contributor guide

Open the contributing guide

Research direction

Start by locating the new file watcher implementation and the consolidation path referenced in the issue. Trace the sequence that removes existing watchers and creates the consolidated watcher, then identify or add coverage for the notification gap. Done means consolidation does not leave a window where file changes can go unreported.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
devtools
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.