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

Add failing test and fix attempt #11798

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

Seldaek
Copy link
Member

@Seldaek Seldaek commented Jan 17, 2024

Refs sebastianbergmann/phpunit#5661

Providers which are not required by anything should not be installable. However the only way to fix this is to make every package require back everything that requires it, to avoid dangling providers being selected.

This may use up too much memory and CPU to be worth fixing, but we'll try here.. This first attempt works for the new test but it does break other things:

  • removal of unused packages, I assume because they now have back-requires keeping them selected
  • detection of dev require vs require, also probably due to the back requires making it all appear to be non-dev requires

@Seldaek Seldaek modified the milestones: 2.6, 2.8, 2.7 Feb 7, 2024
@Seldaek Seldaek modified the milestones: 2.7, 2.8 Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant