-
-
Notifications
You must be signed in to change notification settings - Fork 133
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
Fix CI for php 8 #301
Fix CI for php 8 #301
Conversation
You'll find some thoughts at #247 |
@nicolas-grekas I‘d like to get to the point that we can actually solve #247. Right now, the nightly run ist just plain dead. :-/ |
Let's wait for #303. |
8973635
to
92f299c
Compare
I've rebased the PR on #303. |
92f299c
to
49b402c
Compare
Current state of the PR: The tests should run on php 8 with PHPUnit 9.4. Forget what I wrote. We should fix that listener. |
49b402c
to
2212147
Compare
🤞🏻 The listener should work now. |
2212147
to
c590cdb
Compare
This PR was merged into the 1.20-dev branch. Discussion ---------- GitHub Actions This PR ports our CI to GitHub Actions. While Travis still considers if it should boot the VMs for our CI, the run on GH Actions has been completed already. * This PR includes #303, so please ignore the first commit when reviewing * The PHP 8 run fails (as it does on Travis). I'll try to resolve this in #301. Commits ------- dd2e19f GitHub Actions
c590cdb
to
7fdf16f
Compare
Thank you @derrabus. |
Our tests currently don't work on php 8. I'd like to give it a try to fix this.