-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Update Cake to 0.19.4 #50
Comments
This was actually a mistake, and that dependency shouldn't be there. I believe that @patriksvensson is planning on correcting that in the 0.18.0 release. |
Ah, okay. In that case, we have a couple of options:
Thoughts @gep13 and/or @patriksvensson ? I had a quick start on this last night and I can definitely take option 3 if we want, but I'm not sure I like the inconsistency of option 2... |
My gut feel would be to wait until 0.18.0. I believe that @devlead wanted to get that out sooner rather than later, and then start working on getting a few things working for the Visual Studio 2017 release and moving towards a single scripting engine. |
Fair enough, I might also leave it for now then. Especially since Visual Studio 2017 support might not be quite so straightforward for this 😆 I'll keep an eye out for 0.18.0 and pick this up again then. |
This will also involve updating xUnit.NET dependencies in testing template since Cake.Testing also now depends on beta xUnit packages.
The text was updated successfully, but these errors were encountered: