-
Notifications
You must be signed in to change notification settings - Fork 803
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
Add VSConfig file #6622
Add VSConfig file #6622
Conversation
This is a great step for first time contributors. Is this as minimal as possible? I see e.g. TypeScript and Docker in there, chose (and any others) be removed if they’re not necessary? |
Mea culpa, included some other stuff I installed. |
"Microsoft.Net.ComponentGroup.DevelopmentPrerequisites", | ||
"Microsoft.Component.MSBuild", | ||
"Microsoft.VisualStudio.Component.TextTemplating", | ||
"Microsoft.VisualStudio.Component.SQL.CLR", |
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.
Note - yes, this is technically required
"Microsoft.Net.Component.4.7.2.TargetingPack", | ||
"Microsoft.Net.ComponentGroup.DevelopmentPrerequisites", | ||
"Microsoft.Component.MSBuild", | ||
"Microsoft.VisualStudio.Component.TextTemplating", |
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.
Yes, this is technically required - legacy .NET projects take a dependency on this for the property pages
|
Yay flaky CI. Let's get this in before the machines get angry at me! |
Fixes #6581
This should allow someone to open VisualFSharp.sln in VS without knowing exactly what to install and be prompted by VS to install the correct components.
See this blog post for more info: https://devblogs.microsoft.com/setup/configure-visual-studio-across-your-organization-with-vsconfig/