-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Merge the new UnitTesting solution crawler into Main #64368
Conversation
Remove unneeded SolutionCrawler reasons that UnitTesting is not using.
…/RemoteLegacySolutionEventsAggregationService.cs
…/RemoteLegacySolutionEventsAggregationService.cs
…nEvents Merge main to features/solutionEvents
…sWorkspaceEventListener.cs Co-authored-by: Gen Lu <genlu@users.noreply.github.com>
Wait until an analyzer is registered before streaming workspace events over to OOP
…nEvents Merge main to features/solutionEvents
…nEvents Merge main to features/solutionEvents
…nEvents Merge main to features/solutionEvents
Update to use new internal API
…nEvents Merge main to features/solutionEvents
@tmat Please review. This is merging all the solution-crawler work done in the feature branch back into main :) |
…nEvents Merge main to features/solutionEvents
@tmat review please :) |
private InvocationReasons(ImmutableHashSet<string> reasons) | ||
=> _reasons = reasons; | ||
public InvocationReasons(ImmutableHashSet<string> reasons) | ||
=> _reasons = reasons ?? ImmutableHashSet<string>.Empty; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -19,8 +19,8 @@ namespace Microsoft.CodeAnalysis.ExternalAccess.UnitTesting.Api | |||
public static readonly UnitTestingInvocationReasonsWrapper PredefinedSyntaxChanged = new(PredefinedInvocationReasons.SyntaxChanged); | |||
public static readonly UnitTestingInvocationReasonsWrapper PredefinedProjectConfigurationChanged = new(PredefinedInvocationReasons.ProjectConfigurationChanged); | |||
public static readonly UnitTestingInvocationReasonsWrapper PredefinedDocumentOpened = new(PredefinedInvocationReasons.DocumentOpened); | |||
public static readonly UnitTestingInvocationReasonsWrapper PredefinedDocumentRemoved = new(PredefinedInvocationReasons.DocumentRemoved); | |||
public static readonly UnitTestingInvocationReasonsWrapper PredefinedDocumentClosed = new(PredefinedInvocationReasons.DocumentClosed); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bool ShouldReportChanges(SolutionServices services); | ||
|
||
ValueTask OnWorkspaceChangedAsync(WorkspaceChangeEventArgs args, CancellationToken cancellationToken); | ||
#if false // Not used in unit testing crawling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{ | ||
internal interface INewUnitTestingIncrementalAnalyzerImplementation | ||
{ | ||
#if false // Not used in unit testing crawling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shyamnamboodiripad this is now merged. PTAL when you can and see about moving onto this :) LMK if you run into any issues. Thanks! |
No description provided.