hardkoded / puppeteer-sharp

Headless Chrome .NET API
https://www.puppeteersharp.com
MIT License
3.39k stars 441 forks source link

Migrate from AppVeyor to Github actions #1836

Closed kblok closed 2 years ago

kblok commented 3 years ago

It would be great if we could get out of appveyor.

kgar commented 3 years ago

I'm starting my study of Github Actions right now.

Do you have a preference on how the new setup should work?

kblok commented 3 years ago

I would try to mimic https://github.com/microsoft/playwright-dotnet and or https://github.com/puppeteer/puppeteer

venky8951 commented 2 years ago

Hi @kblok I Have been working on GitHub Actions for a while in my organization can I pick this up ? I have tried out Appveyor's Linux CI and was able to build you can checkout in my forked repo it fails for dotnet test I'm not really sure why, any help is appreciated

If I can resolve this I would like to Migrate AppVeyor's Demo project as well to GitHub Actions

kblok commented 2 years ago

Sure you can! I think it would be nice if we could bump everything to .NET 6 before that. I think it will make your/our life easier.

kblok commented 2 years ago

@venky8951 our tests are now on net 6 :)

venky8951 commented 2 years ago

Hi @kblok I have completed migrating Demo and linux appveyor files in .NET 6 -> Actions but while running tests you can see it fails 7 initially and then in next iteration it passes, any suggestions (are these long running tests and we need to configure some time ?) here is the logs -> https://github.com/venky8951/puppeteer-sharp/runs/4813258463?check_suite_focus=true

kblok commented 2 years ago

@venky8951 are you setting up the SSL certificate?

venky8951 commented 2 years ago

@venky8951 are you setting up the SSL certificate?

Yes Im doing it while Installing Dependencies -> ( dotnet dev-certs https -ep lib/PuppeteerSharp.TestServer/testCert.cer)

kblok commented 2 years ago

Let's force -f net6.0 on linux :)

venky8951 commented 2 years ago

Let's force -f net6.0 on linux :)

Thank you! this seem to be working but one test fails due to this stack trace at PuppeteerSharp.Tests.CoverageTests.JSCoverageTests.ShouldReportScriptsThatHaveNoCoverage() in /home/runner/work/puppeteer-sharp/puppeteer-sharp/lib/PuppeteerSharp.Tests/CoverageTests/JSCoverageTests.cs:line 129

Here are results ->https://github.com/venky8951/puppeteer-sharp/runs/4818304083?check_suite_focus=true

if its okay I can create a PR? ,my fork is 60+ commits ahead ;) (Due to the fact the we need to test these actions on GitHub itself) hope that wont create problems and gets resolved in squash as it happened last time

kblok commented 2 years ago

Yeah. Let's create a PR :)