Skip to content

Conversation

SimonCropp
Copy link
Contributor

so we can use c#11

@SimonCropp
Copy link
Contributor Author

@martincostello do we still want to keep the appveyor build?

@martincostello
Copy link
Member

My views are expressed in #980 (comment) 😃

@joelhulen
Copy link
Member

With Martin planning to move from AppVeyor to GitHub Actions tomorrow, let's hold off on merging this until the new CI/CD process successfully builds.

@martincostello
Copy link
Member

I still need to work out a NuGet publish flow, but if you rebase onto main you should be good to get a clean xplat CI with coverage now.

@codecov-commenter
Copy link

codecov-commenter commented Jan 7, 2023

Codecov Report

Merging #984 (fff7fb6) into main (977f3e1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #984   +/-   ##
=======================================
  Coverage   71.17%   71.17%           
=======================================
  Files         137      137           
  Lines        3768     3768           
  Branches      764      764           
=======================================
  Hits         2682     2682           
  Misses        875      875           
  Partials      211      211           
Flag Coverage Δ
linux 71.17% <ø> (ø)
macos 71.17% <ø> (ø)
windows 71.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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.

4 participants