Fare icon indicating copy to clipboard operation
Fare copied to clipboard

Add AppVeyor CI

Open zvirja opened this issue 7 years ago • 3 comments
trafficstars

We definitely need CI for this project, otherwise it's very dangerous to merge the PRs. Everything is ready for CI (configuration file is already present), you need to do a couple of things only:

  • Go to AppVeyor and create a build for this repo (a few clicks).
  • If you wish to enable automatic release after you create a tag (via GitHub UI), please update the appveyor.yml file and specify the NuGet API Key (encrypted) to the NUGET_API_KEY variable.

zvirja avatar Apr 12 '18 13:04 zvirja

I always run a build on my local before merging a PR, plus this repo didn't receive that many PRs until now, but yeah, this can be useful :+1:

moodmosaic avatar Apr 12 '18 13:04 moodmosaic

I think we need something like this just to be able to grab the generated NuGet Packages and publish them. Personally, I no longer use Visual Studio and having troubles running the build scripts as-is from #30.

Even on Windows, with .NET Core installed, build scripts don't seem to work as per #42.

moodmosaic avatar Jul 17 '18 05:07 moodmosaic

Answered in #42 - build scripts should be rewritten if VS installation is missing, so we use dotnet instead of MsBuild

zvirja avatar Jul 18 '18 09:07 zvirja