search

socketry / nio4r

Cross-platform asynchronous I/O primitives for scalable network clients and servers.
Other
965 stars 86 forks source link

Add Actions CI, remove redundant Travis & AppVeyor jobs #234

Closed MSP-Greg closed 4 years ago

MSP-Greg commented 4 years ago

There's a lot of Actions jobs, but it's not like CI is running that often...

MSP-Greg commented 4 years ago

Damn, Keep forgetting that in repos without Actions, PR's don't run Actions.

See https://github.com/MSP-Greg/nio4r/actions/runs/33731306

tarcieri commented 4 years ago

Nice! Let's give it a try

tarcieri commented 4 years ago

@MSP-Greg mind opening another one to remove the Travis and Appveyor configs?

MSP-Greg commented 4 years ago

@tarcieri

Up to you.

  1. Ruby head just got added to Actions, but I haven't had a chance to run it.

  2. Actions is a PITA for 'allow failure' jobs, one has to click to each job (so one can see the steps) to see if the job failed.

So, I left a few jobs in each that either weren't supported (32 bit on Windows, Ubuntu 14.04 on Travis), along with allow failure jobs.

Lastly, see https://github.com/socketry/nio4r/runs/421030870, there is now a totally new UI for CI, so maybe lets see how this affects everything...