Skip to content

switch from AppVeyor to GitHub Actions to run tests on Windows #58721

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

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Oct 30, 2024

Q A
Branch? 5.4
Bug fix? no
New feature? no
Deprecations? no
Issues
License MIT

In #58370 I tried hard but failed to get our test suite running with PHPUnit 11 on AppVeyor. @wouterj did a great job reviving an old branch and got it working on GitHub Actions in #58699. I picked his commit and rebased it on the 5.4 branch so that we are able to have a consistent CI config throughout all maintained branches.

@carsonbot carsonbot added this to the 5.4 milestone Oct 30, 2024
@chadyred
Copy link
Contributor

Oh, sounds good guys 🙏 I discovered AppVeyor by searching why tests go away sometimes for no reason ^^

@OskarStark
Copy link
Contributor

You did an amazing job, thank you very much. 🖖

Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

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

to pptimize the display in the GH UI :)

@nicolas-grekas
Copy link
Member

Thank you @wouterj and @xabbuh \o/

@nicolas-grekas nicolas-grekas merged commit 2c7c4ba into symfony:5.4 Nov 6, 2024
10 of 12 checks passed
@wouterj
Copy link
Member

wouterj commented Nov 6, 2024

Wohoo! 🚀 Thanks for all the debugging work @xabbuh!

Next step I will experiment with: ForEach -Parallel to do the similar parallelization as we do in the Linux builds. Managed to get powershell running in Docker so I can properly test this locally.

@xabbuh xabbuh deleted the gha-windows branch November 6, 2024 14:49
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.

7 participants