Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@andyfaff
Copy link
Member

Removes two windows jobs from Azure Pipelines, transferring them to GHA.

@andyfaff andyfaff changed the title CI: remove two windows jobs from Azure pipelines CI: remove (mainly windows) jobs from Azure pipelines Nov 15, 2025
@rgommers
Copy link
Member

Sounds great, finishes off the migration we discussed in gh-24410. Thanks for working on this @andyfaff! Having a closer look now.

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks pretty good, thanks @andyfaff! And the jobs are way faster on GHA than they were on Azure.

I made some inline comments that would be good to address. In addition, it'd be nice to make the job names more consistent, because it's now hard to spot the difference between the two platforms. I'd propose a fixed pattern: compiler, platform, fast/full, BLAS. Which would yield:

  • Clang-cl, x86-64, fast, openblas32
  • MSVC, x86, fast, no BLAS
  • MSVC, arm64, fast, no BLAS
  • MSVC, x86-64, full, openblas64
  • MSVC, x86-64, fast, openblas32

@andyfaff
Copy link
Member Author

I'm going to close this PR now to do further work on my fork. I'll reopen when it's ready to be evaluated again.

@andyfaff andyfaff closed this Nov 16, 2025
@andyfaff andyfaff reopened this Nov 16, 2025
@andyfaff andyfaff requested a review from rgommers November 16, 2025 01:11
Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update Andrew, LGTM now modulo one very minor last thing.

@rgommers rgommers merged commit e468d88 into numpy:main Nov 16, 2025
74 checks passed
@rgommers
Copy link
Member

In it goes, thanks Andrew!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants