Skip to content

Conversation

@WyriHaximus
Copy link

No description provided.

@frank9999
Copy link
Contributor

Thanks for this PR! I also wanted to add PHP 8.1 and 8.2 to CI :) I also noticed that PHP 7.2 and higher has pipelines failing. Added an issue #927

@WyriHaximus
Copy link
Author

@frank9999 Just waiting for Chris to finish the RFC package before he looks at this. Already had a chat with him about this in private.

@cboden cboden changed the base branch from master to v0.4.4 December 12, 2021 16:58
@cboden
Copy link
Member

cboden commented Dec 12, 2021

Can you rebase your branch please? This should fix the failing tests.

@cboden cboden modified the milestone: 0.4.4 Dec 12, 2021
@cboden cboden deleted the branch ratchetphp:0.4.x December 14, 2021 00:14
@cboden cboden closed this Dec 14, 2021
@cboden cboden reopened this Dec 14, 2021
@cboden cboden changed the base branch from v0.4.4 to master December 14, 2021 00:22
@WyriHaximus
Copy link
Author

@cboden Just rebased and push force with lease

@cboden
Copy link
Member

cboden commented Dec 16, 2021

@WyriHaximus Anything we can do to fix the failing tests on 8.x?

@WyriHaximus
Copy link
Author

@cboden yeah will fix that as well 👍

@WyriHaximus
Copy link
Author

This is going to be a fun upgrade 😂

@cboden
Copy link
Member

cboden commented Dec 16, 2021

Would putting the phphunit versions in the opposite order, or perhaps only 4.x and 7.x satisfy some problems?

@WyriHaximus
Copy link
Author

Not sure, locally it only installs 4 some how, even though I'm running PHP 8.0

@WyriHaximus
Copy link
Author

@cboden Feel free to run, but tests will fail currently. Working on fixing them/

@WyriHaximus WyriHaximus changed the title Test on PHP 8.0 and 8.1 Test on PHP 8.0, 8.1, and 8.2 Jan 16, 2023
@WyriHaximus
Copy link
Author

Updated PR to also include PHP 8.2

@Tofandel
Copy link

This should also include #1060 as the warnings are not fixed in the tests

@PaulRotmann
Copy link
Contributor

Hey! Thanks for putting this together 🙌

As part of our cleanup initiative (#1100), we're closing this since we've merged #1094 & #1096 which should cover the functionality.

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.

6 participants