-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Add in VS 2022 support. #5956
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 in VS 2022 support. #5956
Conversation
Codecov Report
@@ Coverage Diff @@
## main #5956 +/- ##
==========================================
+ Coverage 68.19% 68.23% +0.04%
==========================================
Files 1142 1142
Lines 242367 242803 +436
Branches 25355 25385 +30
==========================================
+ Hits 165287 165686 +399
- Misses 70400 70421 +21
- Partials 6680 6696 +16
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Added a little comment. LGTM otherwise.
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.
Do we have any reason to keep support for VS2015 and VS2017 around?
This is just for building ML.NET. Do you think there is a need to remove the ability to build it on older versions of visual studio? I'm not sure if people are still using older versions or not. Is there a way to see? |
There is not, it is not a big deal, we just in dotnet/runtime remove support for the old ones where the TFMs we use are not even supported. |
Added in support for VS 2022 in the native build.
Also added documentation stating the version of CMake needed for 2022.