-
Notifications
You must be signed in to change notification settings - Fork 46
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
Convert to a VS2017 project #31
Conversation
…on goes smoothly
… only installed VS2017 and not the 7-Mar-2017 SDK
…ig so everyone gets the new version of cake
global.json
Outdated
@@ -0,0 +1,6 @@ | |||
{ | |||
"projects": [ "source" ], |
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.
How did u find out about "projects" ? It is not documented https://docs.microsoft.com/en-us/dotnet/articles/core/tools/global-json
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.
Apparently u don't need it, see dotnet/docs#996 (comment)
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.
My bad, copy pasta :)
… now included by default. The nuspec wasn't used.
@@ -2,243 +2,118 @@ | |||
// TOOLS | |||
////////////////////////////////////////////////////////////////////// | |||
#tool "nuget:?package=GitVersion.CommandLine&version=4.0.0-beta0007" | |||
#addin "MagicChunks" | |||
#tool "nuget:?package=NUnit.ConsoleRunner"&version=3.6.1 |
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.
@droyad given that we have upgraded to xunit, I don't think we need this anymore
No description provided.