Skip to content

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

Merged
merged 19 commits into from
Jan 24, 2017

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Jan 20, 2017

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.

@daviwil
Copy link
Contributor Author

daviwil commented Jan 20, 2017

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 :)

@daviwil daviwil added this to the 0.9.1 milestone Jan 20, 2017
@daviwil daviwil force-pushed the daviwil/dotnet-cli-redux branch 18 times, most recently from 82b737d to 497e658 Compare January 21, 2017 18:31
@daviwil
Copy link
Contributor Author

daviwil commented Jan 21, 2017

Looks like the AppVeyor build is finally working! I've got a few more things to finish up before merging.

Remaining Work

  • Clean up commits
  • Set up a version manipulation task, feed it the version from AppVeyor
  • Run dotnet pack and upload packages as artifacts
  • File bug to track work needed to move over to new VS 2017 image on AppVeyor so that their test logger works (Visual Studio 2017 RC appveyor/ci#1179)
  • Build PSES from within vscode-powershell project when in VS Code. Should be able to change code in PSES and then hit F5 in vscode-powershell to see the new changes get used
  • Write new developer docs for building the project

@daviwil daviwil force-pushed the daviwil/dotnet-cli-redux branch from 5731bc5 to e67344c Compare January 22, 2017 00:02
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.
@daviwil daviwil force-pushed the daviwil/dotnet-cli-redux branch from e67344c to 41ac945 Compare January 22, 2017 00:08
@daviwil daviwil force-pushed the daviwil/dotnet-cli-redux branch from 9bf7364 to cd3a1f4 Compare January 22, 2017 02:18
@daviwil daviwil changed the title WIP: Convert all projects to use dotnet cli preview4 tools WIP: Convert all projects to use .NET CLI preview4 tools Jan 22, 2017
@daviwil daviwil changed the title WIP: Convert all projects to use .NET CLI preview4 tools Convert all projects to use .NET CLI preview4 tools Jan 22, 2017
@daviwil daviwil mentioned this pull request Jan 22, 2017
12 tasks
@daviwil daviwil force-pushed the daviwil/dotnet-cli-redux branch 2 times, most recently from 467af68 to c0bf13a Compare January 22, 2017 20:59
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.
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.
@daviwil
Copy link
Contributor Author

daviwil commented Jan 24, 2017

Everything works well with the new VS Code integration, merging this!

@daviwil daviwil merged commit b6c10be into develop Jan 24, 2017
@daviwil daviwil deleted the daviwil/dotnet-cli-redux branch January 24, 2017 00:14
@daviwil daviwil modified the milestones: 0.9.1, 0.10.0 Mar 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants