-
Notifications
You must be signed in to change notification settings - Fork 25
Support for PHPUnit 10 and Laravel 10.x #47
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
base: develop
Are you sure you want to change the base?
Conversation
Add support for PHPUnit 10
The build with PHP 8.1 fails on travis. This is related to an outdated composer.lock file in this repository. I think we might need to do a bit of cleaning here before adding support to php 8.1. |
@ventrec Ah yeah, we'll need to switch away from listeners and to use the new Event/subscribers system for PHPUnit 10. I've made progress converting the package but I'm not sure how to override the output of PHPUnit. The |
FYI: I'm working on a PHPUnit pretty output based on https://pestphp.com/docs/installation. I'm using PHPUnit 10 and got an working example... |
@robiningelbrecht How close of a replacement to this package is it and when will it be ready? |
@Braunson it's ready and based on Pest PHPs output. Not sure if I should post it here? It's a separate repo |
Just check my repo's it's in the list :) |
@robiningelbrecht I think it's this one, right? https://github.com/robiningelbrecht/phpunit-pretty-print I love Pest output... so I'm on it :) |
@TomasVotruba indeed it is :) |
@robiningelbrecht I see :) thanks. I'm just trying it and it seems it can be only enabled in every run in xml. E.g.: vendor/bin/phpunit --output-formatter=PrettyOne Any idea how to achieve this? |
@TomasVotruba no, not right now. Could you create a feature request in the issue queue? I'll look into it. Let's move the discussion there |
Uh oh!
There was an error while loading. Please reload this page.