-
Notifications
You must be signed in to change notification settings - Fork 237
Convert all projects to use .NET CLI preview4 tools #351
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
Conversation
It finally works! I'm really pleased with the new MSBuild-based toolset. I'll get the AppVeyor build and tests passing again today, hopefully should get far enough along that we can start using this for day to day development. Best part is that we can now use VS Code for PSES development :) |
82b737d
to
497e658
Compare
Looks like the AppVeyor build is finally working! I've got a few more things to finish up before merging. Remaining Work
|
5731bc5
to
e67344c
Compare
This change migrates all existing projects over to MSBuild projects that work with preview4 of the dotnet cli tools. This allows us to build our Full- and CoreCLR code with a single set of projects, even our our CI server.
This change establishes an Invoke-Build script for building and testing the project, the VS Code tasks that will launch the build script, and an AppVeyor configuration that can run the build script in a CI build.
e67344c
to
41ac945
Compare
9bf7364
to
cd3a1f4
Compare
467af68
to
c0bf13a
Compare
This change updates the .build.ps1 script to point at the 1.0.0-rc3 release of the .NET SDK so that we can make use of updated test command that returns the correct exit code when tests fail. The build script is also updated to pull the SDK version from global.json. This file is also used to tell the dotnet CLI tool which SDK version to use.
There are some issues with the rc3/rc4 releases of the SDK around NuGet packaging, constant failures. Downgrading to preview4 until those are investigated and resolved.
bd566d1
to
bd3b3f6
Compare
Apparently the default build script name for Invoke-Build is not .build.ps1, it's actually FolderName.build.ps1. That works out better for developers on Linux and macOS because the build script will not be hidden in directory listings.
See #353 for more details on why this doesn't work currently.
4d40650
to
bad9870
Compare
Everything works well with the new VS Code integration, merging this! |
This change migrates all existing projects over to MSBuild projects
that work with preview4 of the dotnet cli tools. This allows us to build
our Full- and CoreCLR code with a single set of projects, even our our CI
server.