Skip to content

WIP: Convert projects to project.json #282

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

Closed
wants to merge 4 commits into from
Closed

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Aug 29, 2016

This change converts all existing projects to the new project.json format which can build both full CLR and CoreCLR binaries for these assemblies. Tracked by issue #281.

Starting this pull request early so I can test AppVeyor build/test runs.

Things to do

  • Rename all NanoServer compiler directives to CoreCLR
  • Get debugging working in VS Code
  • Create .sln file for opening project in Visual Studio (hopefully just for debugging)
  • dotnet pack step to create NuGet packages
  • Update PowerShellEditorServices.psm1 to load binaries from updated paths in module
  • Copy binary output to module path
  • Install open-source PowerShell on AppVeyor to run build and tests
  • Get xUnit tests running from inside of PowerShell with real-time output
  • Upload NuGet artifacts plus a zip of the module files
  • Update release scripts to handle new NuGet packages
  • Remove all old .csproj files and update .sln
  • Make sure included .ps1 files get bundled with assemblies (resgen?)

@daviwil
Copy link
Contributor Author

daviwil commented Aug 29, 2016

Looks like the WSMan package dependency is also a problem on Windows, going to disable net451 build until that package gets shipped.

Issue tracked here: PowerShell/PowerShell#2065

@daviwil daviwil force-pushed the daviwil/dotnet-cli branch 6 times, most recently from c3dcfb4 to 6d198cd Compare August 30, 2016 22:38
This change converts all existing projects to the new project.json format which can build both full CLR and CoreCLR binaries for these assemblies.
This change makes some spot fixes to alleviate compiler warnings that were being produced in the Protocol project.
@daviwil daviwil force-pushed the daviwil/dotnet-cli branch 2 times, most recently from 69768cc to e7e4f75 Compare September 3, 2016 00:23
@daviwil
Copy link
Contributor Author

daviwil commented Jan 22, 2017

This PR isn't needed anymore now that #351 is a success.

@daviwil daviwil closed this Jan 22, 2017
@TylerLeonhardt TylerLeonhardt deleted the daviwil/dotnet-cli branch December 14, 2018 02:45
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