Skip to content
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

Modernize test suite #148

Merged

Conversation

WyriHaximus
Copy link
Member

Our current test suite has been around for a while and to keep it in
shape for the future we need to modernize is one in a while.

@WyriHaximus WyriHaximus added this to the v3.0.0 milestone Oct 7, 2019
@WyriHaximus WyriHaximus force-pushed the rector-phpunit-upgrades branch 5 times, most recently from 55eccba to c4543e9 Compare October 7, 2019 21:20
Our current test suite has been around for a while and to keep it in
shape for the future we need to modernize is one in a while.

With the increase to PHP 7.1+ in reactphp#149 we can also bump the PHPUnit
version to 7.
@WyriHaximus WyriHaximus force-pushed the rector-phpunit-upgrades branch from c4543e9 to b41ca0f Compare October 7, 2019 21:22
@WyriHaximus WyriHaximus changed the title [WIP] Modernize test suite Modernize test suite Oct 7, 2019
@WyriHaximus WyriHaximus requested review from jsor and clue October 7, 2019 21:22
@WyriHaximus
Copy link
Member Author

@jsor @clue ok PR is reqdy for review now

Copy link
Member

@clue clue left a comment

Choose a reason for hiding this comment

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

No objections (personally, I tend to avoid importing global classes, but I'm fine with this either way) 👍

@jsor jsor merged commit 15036d2 into reactphp:master Oct 8, 2019
@WyriHaximus WyriHaximus deleted the rector-phpunit-upgrades branch October 8, 2019 18:20
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.

4 participants