Skip to content

Fix checks for phpunit releases on Composer 2 #37607

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

Merged
merged 1 commit into from
Jul 23, 2020

Conversation

colinodell
Copy link
Contributor

@colinodell colinodell commented Jul 19, 2020

Q A
Branch? 3.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #37601
License MIT
Doc PR n/a

simple-phpunit has a core assumption that any version of PHPUnit without a stable release will only have one dev version returned by Composer. Per #37601, it's possible for Composer 2 to list more than one dev version. This breaks that assumption and therefore prevents the installation of 9.3.* (which is needed for testing on PHP 8).

The fix implemented here is to remove any versions containing dev- or -dev from the list of possible versions to see if any stable versions remain.

@GrahamCampbell
Copy link
Contributor

Related: composer/composer#9054.

@colinodell colinodell changed the title Fix checks for phpunit releases on Composer 2 (resolves #37601) Fix checks for phpunit releases on Composer 2 Jul 19, 2020
@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jul 22, 2020
Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, just some minor questions and good to go on my side

@colinodell colinodell force-pushed the fix-phpunit-bridge-composer-2 branch from f64114e to 2bb3f08 Compare July 22, 2020 22:00
@fabpot
Copy link
Member

fabpot commented Jul 23, 2020

Thank you @colinodell.

@fabpot fabpot merged commit 0eafc01 into symfony:3.4 Jul 23, 2020
nicolas-grekas added a commit that referenced this pull request Jul 23, 2020
This PR was squashed before being merged into the 4.4 branch.

Discussion
----------

Require PHPUnit 9.3 on PHP 8

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| License       | MIT

The first version of PHPUnit to support PHP 8.0 is PHPUnit 9.3.

---

Depends on #37607. Related to composer/composer#9054.

Commits
-------

54b13c0 Require PHPUnit 9.3 on PHP 8
@colinodell colinodell deleted the fix-phpunit-bridge-composer-2 branch July 23, 2020 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants