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
Tweaking phpunit post-install message to give more information/context #811
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request passes validation.
This is because, it *does* look a little odd to remove phpunit/phpunit and require phpunit. This is more descriptive
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request passes validation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request passes validation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request passes validation.
The previous message appears to some people like Symfony is trying to "force" people what to do or to replace PHPUnit.
In reality, the goal is to make sure the user is aware that - to follow the "documented path" - the bridge is available. But if they still want to install PHPUnit, that's totally fine.
Related to: symfony/symfony#27289 and https://twitter.com/Ocramius/status/1293169839582973952