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

Please consider reverting require: "phpunit/phpunit" in composer.json #86

Open
FelixJacobi opened this issue Aug 26, 2024 · 0 comments
Open

Comments

@FelixJacobi
Copy link

FelixJacobi commented Aug 26, 2024

add support for phpunit 11 (#79) · SymfonyTest/SymfonyConfigTest@59f4d3b · GitHub added "require": "phpunit/phpunit to the composer.json.

This is very impractical in some situations. We have several composer.json files in our project:

  • tools/.phpunit/composer.json (PHPUnit)
  • composer.json (actual project)

We do this because there are conflicts between PHPUnit and other tools (like Psalm) for their respective dependencies.

We had to add this library here to the main composer.json because it has Symfony dependencies and we would have duplicate Symfony packages otherwise.

Due to this dependency, a PHPUnit is now installed in the main project, which causes conflicts.

I would assume that the explicit dependency in require is not needed, since you will certainly not use this library without PHPUnit and it is therefore always present.

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

No branches or pull requests

1 participant