-
Notifications
You must be signed in to change notification settings - Fork 77
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
Drop support for VS2019 #5444
Drop support for VS2019 #5444
Conversation
@@ -33,7 +33,6 @@ namespace SonarLint.VisualStudio.IssueVisualization.Security.OpenInIDE | |||
[PackageRegistration(UseManagedResourcesOnly = true, AllowsBackgroundLoading = true)] | |||
[Guid("5C38BF5B-F6D1-4232-A5B5-5664B6A19A91")] | |||
// We want to start listening as soon as the IDE starts. | |||
// Note: in VS2019, this will be when the new modal start page appears. |
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.
It was about vs17 to vs19. The info is still relevant. Change to Since vs19,...
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.
I think it's not that important since this is the default behaviour info about this change is not interesting anymore and I think it should've been removed when dropping 2017.
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.
I think it's valuable for someone to know that the shell initializes before you actually select any project, however this comment is not the most noticeable place, so up to you
src/Roslyn.Suppressions/Roslyn.Suppressions/SupportedSuppressionBuilder.g.cs
Outdated
Show resolved
Hide resolved
src/TypeScript/NodeJSLocator/LocationProviders/BundledNodeLocationsProvider.cs
Outdated
Show resolved
Hide resolved
|
7fdfc25
into
feature/hardening-may-2024
Part of #5345