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

[PhpUnitBridge] Use COMPOSER_BINARY env var if available #36566

Merged
merged 1 commit into from Apr 25, 2020

Conversation

fancyweb
Copy link
Contributor

Q A
Branch? 3.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets #36401
License MIT
Doc PR -

@fabpot
Copy link
Member

fabpot commented Apr 25, 2020

Thank you @fancyweb.

@fabpot fabpot merged commit e3dc5ef into symfony:3.4 Apr 25, 2020
@fancyweb fancyweb deleted the php-unit-bridge-composer-binary branch April 25, 2020 19:36
This was referenced Apr 28, 2020
nicolas-grekas added a commit that referenced this pull request Jun 22, 2021
…hen using simple-phpunit (Taluu)

This PR was merged into the 4.4 branch.

Discussion
----------

[PhpUnitBridge] fix handling the COMPOSER_BINARY env var when using simple-phpunit

| Q             | A
| ------------- | ---
| Branch?       | 4.4 and next
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #41774
| License       | MIT

Since the renaming of the simple phpunit file (from the shebang to the php file) in #31364, it looks like the change made from #36566 was lost as it seems to have been merged after.

So this PR is an attempt to "restore" that functionnality, as it was considered a bug fix on 3.4 at the time.

poke `@nicolas`-grekas

Commits
-------

9589b52 Reapply the change to allow to set the composer binary path
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.

None yet

5 participants