-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Switch to windows-2022 in CI #18927
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch to windows-2022 in CI #18927
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @shivammathur!
windows-2019 runner will be dropped by GitHub on 2025-06-30.
869008c
to
2657fac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @shivammathur!
I'm generally fine with the PR, but if you prefer to rewrite find-vs-toolset.bat in Powershell, I'm certainly not against that.
Regarding the test failures: almost certainly due to stack unwinding on Windows 2022; @dstogov fixed that with ccc6c0f for PHP-8.4. I don't think that we can backport that, so it might be best to mark the test as XFAIL.
@cmb69 I think we can keep it in batch for consistency. |
Yeah, sure! (Actually, I'm neither happy with complex batch scripts, nor with Powershell scripts.) |
This is a continuation of phpGH-18927 to fix CI for windows-2022
This is a continuation of phpGH-18927 to fix CI for windows-2022
This is a continuation of phpGH-18927 to fix CI for windows-2022
windows-2019 runner will be dropped by GitHub on 2025-06-30.
Ref: actions/runner-images#12045