search

mikeerickson / phpunit-pretty-result-printer

PHPUnit Pretty Result Printer -- make your PHPUnit tests look pretty!
MIT License
1.2k stars 71 forks source link

fix infinite loop on windows fixes #37 #39

Closed peterlupu closed 6 years ago

sorincristea commented 6 years ago

thanks peterlupu! It works for me!

mikeerickson commented 6 years ago

Awesome, thank you very much for the PR. I will get this merged and tagged later this afternoon

mikeerickson commented 6 years ago

FYI, version 0.7.0 has been tagged, merging this PR

thanks peterlupu! It works for me!

JayAhrDe commented 6 years ago

Got 0.17.1 installed and still get that message (without "class does not exist", though: there simply is no explanation in my case.

mikeerickson commented 6 years ago

@jayahrde would you mind removing completely and reinstall. I am puzzled on this end as it works without issue using 0.17.1 on windows 10, PHPUnit 7.1 and php 7.1

JayAhrDe commented 6 years ago

Got to get back to it on my home machine, hope I won't forget.

Zugmaschine commented 6 years ago

Hello, i saw the solution, but i cant fix it. can you describe me, how i have to use that code solution? Is it to delete the green marked code, and add the red marked code at this place? I use Windows 10 and the newes Version of Laravel Boilerplate.

mikeerickson commented 6 years ago

@zugmaschine this should already be available in the latest release. What version do you have installed.

Zugmaschine commented 6 years ago

Hello Mike, thanks for your quick answer. I use the latest version(5.6.34). So i wonder. Do you have a idea?

Regards, Uwe

Am Di., 28. Aug. 2018 um 20:08 Uhr schrieb Mike Erickson < notifications@github.com>:

@Zugmaschine https://github.com/Zugmaschine this should already be available in the latest release. What version do you have installed.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mikeerickson/phpunit-pretty-result-printer/pull/39#issuecomment-416685850, or mute the thread https://github.com/notifications/unsubscribe-auth/Aeno0MX6kqUWpuYp4xcA6xbufxGFcxMpks5uVYcKgaJpZM4S1U5z .

mikeerickson commented 6 years ago

I am referring to the version of the report printer.