search

microsoft / ConcordExtensibilitySamples

Visual Studio Debug Engine Extensibility Samples
Other
121 stars 50 forks source link

Add PR build validation #68

Closed gregg-miskelly closed 2 years ago

gregg-miskelly commented 2 years ago

This PR adds back PR validation to the ConcordExtensibilitySamples repo.

Changes:

  1. Added new PR .yml that builds the projects in this solution. It does so by executing the build script.
  2. Eliminate BuildAndTest.proj in favor of building through build.cmd. BuildAndTest.proj was making it tricky to track down failures because it wasn't acting exactly the same as running each solution independently.
  3. Removed the old build files

Testing:

plnelson commented 2 years ago

Do we need to update readme.md as well to fix the build status icons?

gregg-miskelly commented 2 years ago

Do we need to update readme.md as well to fix the build status icons?

Yes. Sorry, this PR should really be a draft. I haven't gotten it to work yet.