From 3c5e96dc8d15b7f31f7696bc6fe2385bccfe4e65 Mon Sep 17 00:00:00 2001 From: Jason Malinowski Date: Fri, 24 Aug 2018 14:53:20 -0700 Subject: [PATCH] Add a comment for why this is written this way I almost tried to refactor this and then caught myself. --- .../AnalyzersFolderItem/AnalyzersFolderItemProvider.cs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/VisualStudio/Core/SolutionExplorerShim/AnalyzersFolderItem/AnalyzersFolderItemProvider.cs b/src/VisualStudio/Core/SolutionExplorerShim/AnalyzersFolderItem/AnalyzersFolderItemProvider.cs index 369941aca4a1b..a7f4bf7e764f2 100644 --- a/src/VisualStudio/Core/SolutionExplorerShim/AnalyzersFolderItem/AnalyzersFolderItemProvider.cs +++ b/src/VisualStudio/Core/SolutionExplorerShim/AnalyzersFolderItem/AnalyzersFolderItemProvider.cs @@ -21,6 +21,8 @@ namespace Microsoft.VisualStudio.LanguageServices.Implementation.SolutionExplore [Order(Before = HierarchyItemsProviderNames.Contains)] internal class AnalyzersFolderItemProvider : AttachedCollectionSourceProvider { + // NOTE: the IComponentModel is used here rather than importing ISolutionExplorerWorkspaceProvider directly + // to avoid loading VisualStudioWorkspace and dependent assemblies directly private readonly IComponentModel _componentModel; private readonly IAnalyzersCommandHandler _commandHandler; private IHierarchyItemToProjectIdMap _projectMap;