Skip to content

Make result printer compatible with phpUnit version 7 #18

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jimigrunge
Copy link

  • Fixed parameter type hinting
  • Fixed void returns
  • Updated composer settings
  • Updated phpunit xml config to v7

 - Fixed parameter type hinting
 - Fixed void returns
 - Changing composer settings
 - Updated phpunit xml config
@amenk
Copy link

amenk commented Feb 11, 2019

Looks good to me, please merge :-9

@JeroenDeDauw
Copy link
Contributor

PHPUnit 7 support is here https://github.com/JeroenDeDauw/nyancat-phpunit-resultprinter

@malukenho
Copy link

malukenho commented Feb 24, 2020

Thanks @JeroenDeDauw! I'm using your fork <3

May you update it to work with PHPUnit 9 as well?

@JeroenDeDauw
Copy link
Contributor

I just made a new release that adds compat with PHPUnit 8. Getting PHPUnit 9 to work looks rather involved and I do not currently have the time to spend on that. Pull requests welcome though!

@malukenho
Copy link

@JeroenDeDauw May you enable issue creation on your Fork, so we can continue there?

@JeroenDeDauw
Copy link
Contributor

Huh. Did not realize those where disabled. Here you go: https://github.com/JeroenDeDauw/nyancat-phpunit-resultprinter/issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants