Skip to content

Use brokered service for IManagedHotReloadService in VS#83018

Merged
tmat merged 2 commits intodotnet:mainfrom
tmat:UseIManagedHotReloadServiceBrokeredService
Apr 3, 2026
Merged

Use brokered service for IManagedHotReloadService in VS#83018
tmat merged 2 commits intodotnet:mainfrom
tmat:UseIManagedHotReloadServiceBrokeredService

Conversation

@tmat
Copy link
Copy Markdown
Member

@tmat tmat commented Apr 1, 2026

Microsoft Reviewers: Open in CodeFlow

@tmat tmat marked this pull request as ready for review April 2, 2026 21:42
@tmat tmat requested a review from a team as a code owner April 2, 2026 21:42
@tmat
Copy link
Copy Markdown
Member Author

tmat commented Apr 2, 2026

@DustinCampbell ptal

@@ -1,42 +0,0 @@
// Licensed to the .NET Foundation under one or more agreements.
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved to EditorFeatures layer and linked to DevKit

{ RemoteProjectInitializationStatusService.Moniker, new ServiceRegistration(ServiceAudience.Local, null, allowGuestClients: false) },
{ BrokeredServiceDescriptors.SolutionSnapshotProvider.Moniker, new ServiceRegistration(ServiceAudience.Local, null, allowGuestClients: false) },
{ BrokeredServiceDescriptors.DebuggerManagedHotReloadService.Moniker, new ServiceRegistration(ServiceAudience.Local, null, allowGuestClients: false) },
{ BrokeredServiceDescriptors.HotReloadSessionNotificationService.Moniker, new ServiceRegistration(ServiceAudience.Local, null, allowGuestClients: false) },
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removes unused services

@tmat tmat merged commit 3de19a1 into dotnet:main Apr 3, 2026
25 checks passed
@tmat tmat deleted the UseIManagedHotReloadServiceBrokeredService branch April 3, 2026 15:57
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants