-
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
Cleanup in VS generator UI code. #74448
Conversation
@@ -9,83 +9,73 @@ | |||
using Microsoft.VisualStudio.Imaging; |
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.
view with whitespace off.
@@ -9,83 +9,73 @@ | |||
using Microsoft.VisualStudio.Imaging; | |||
using Microsoft.VisualStudio.Imaging.Interop; | |||
|
|||
namespace Microsoft.VisualStudio.LanguageServices.Implementation.SolutionExplorer | |||
namespace Microsoft.VisualStudio.LanguageServices.Implementation.SolutionExplorer; |
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.
FSNs and primary constructors.
@@ -33,11 +33,11 @@ internal abstract class BaseItem : | |||
{ | |||
public virtual event PropertyChangedEventHandler PropertyChanged { add { } remove { } } | |||
|
|||
private readonly string _name; | |||
protected readonly string Name; |
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.
part of the equality contract the derived tuypes check.
@@ -123,7 +123,7 @@ private BulkObservableCollection<BaseItem> CreateDiagnosticAndGeneratorItems(Pro | |||
|
|||
collection.AddRange( | |||
AnalyzerReference.GetGenerators(language) | |||
.Select(g => new SourceGeneratorItem(projectId, g, AnalyzerReference))); | |||
.Select(g => new SourceGeneratorItem(projectId, SourceGeneratorIdentity.Create(g, AnalyzerReference), AnalyzerReference.FullPath))); |
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.
moved SGItem to be pure data.
|
||
ruleSetDocument.Save(pathToRuleSet); | ||
} | ||
public override int GetHashCode() |
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.
equals/gethashcode are used in next PR as part of proper items in a observable-collectoin and virtualizing-tree-view.
src/VisualStudio/Core/Impl/SolutionExplorer/AnalyzerItem/AnalyzerItem.cs
Show resolved
Hide resolved
Formatting In progress In progress In progress In progress Host side' git pus Host side' git pus Host side' git pus Host side' git pus Host side Remote side Accessibility Tests Cast In progress' UI thread Tests Make private Make private Docs Simplify Clear the items Docs fix Reversinos Reversinos revert revert revert revert
d10efa4
to
bedf640
Compare
src/VisualStudio/Core/Impl/SolutionExplorer/DiagnosticItem/CpsDiagnosticItemSourceProvider.cs
Show resolved
Hide resolved
src/VisualStudio/Core/Impl/SolutionExplorer/DiagnosticItem/DiagnosticItem.cs
Outdated
Show resolved
Hide resolved
src/VisualStudio/Core/Impl/SolutionExplorer/DiagnosticItem/DiagnosticItem.cs
Show resolved
Hide resolved
...VisualStudio/Core/Impl/SolutionExplorer/DiagnosticItem/LegacyDiagnosticItemSourceProvider.cs
Show resolved
Hide resolved
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.
ok. will ping once this goes in and i meerge this into the other pr. |
Pulling out cleanup code from #74444