Skip to content

CI: Move appveyor/azure jobs to GitHub actions - #32395

Open
greglucas wants to merge 3 commits into
matplotlib:mainfrom
greglucas:ci/windows-github-actions
Open

greglucas wants to merge 3 commits into
matplotlib:mainfrom
greglucas:ci/windows-github-actions

Conversation

@greglucas

Copy link
Copy Markdown
Contributor

PR summary

The appveyor and azure jobs frequently have issues and take much longer to run. When doing some quick googling it looks like GitHub Actions uses the same underlying infrastructure but may get more resources per each CI job. This is an attempt to consolidate our CI into one common language so we don't have to know what the configuration formats for multiple CI providers is either.

Since Windows had some bespoke things in it I made this a separate workflow file so we don't overwhelm the existing tests yaml with conditions of if windows...

AI Disclosure

I had AI do the conversion of the configuration files from appveyor/azure into GitHub Actions and read through the output before PRing. I am not fluent in powershell, but it all looks reasonable.

@greglucas greglucas added the CI: testing CI configuration and testing label Sep 25, 2026
@github-actions github-actions Bot added the Documentation: devdocs files in doc/devel label Sep 25, 2026
@iccir

iccir commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

I asked about this in the weekly meeting yesterday! There was mention that we keep Appveyor around because it tests conda, although I don't think anyone is enthusiastic about the frequent hangs.

A lot of the discussion went over my head though as I'm still new to the whole Python ecosystem.

@greglucas

Copy link
Copy Markdown
Contributor Author

I asked about this in the weekly meeting yesterday! There was mention that we keep Appveyor around because it tests conda, although I don't think anyone is enthusiastic about the frequent hangs.

I did port the conda test as well and now it looks like there must be some architectural difference (or running these tests now when we weren't before) because there are two minor 3d plot image comparison changes.

@rcomer rcomer linked an issue Sep 25, 2026 that may be closed by this pull request
@greglucas
greglucas force-pushed the ci/windows-github-actions branch from a937204 to 8eaedaf Compare September 25, 2026 15:35
@greglucas

Copy link
Copy Markdown
Contributor Author

Not sure why the codecov/patch is reporting 0% diff, clicking into it does show the lines are being hit/missed same as before as expected.

Looking at the timings, all CI jobs finished in 23 minutes, versus ~33 minutes (or more) on the previous azure/appveyor versions.

@iccir iccir mentioned this pull request Sep 29, 2026
3 tasks done

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: testing CI configuration and testing Documentation: devdocs files in doc/devel topic: mplot3d

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MNT]: move Windows tests to GitHub Actions?

2 participants