Skip to content

[Testing] [PHPUnitBridge] Docs for the new SYMFONY_PHPUNIT_REQUIRE env variable #14913

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
Feb 8, 2021

Conversation

acasademont
Copy link
Contributor

@carsonbot carsonbot changed the title [PhpUnitBridge] Docs for the new SYMFONY_PHPUNIT_REQUIRE env variable [PHPUnitBridge] Docs for the new SYMFONY_PHPUNIT_REQUIRE env variable Feb 1, 2021
@OskarStark OskarStark added the Waiting Code Merge Docs for features pending to be merged label Feb 2, 2021
@carsonbot carsonbot added this to the next milestone Feb 2, 2021
fabpot added a commit to symfony/symfony that referenced this pull request Feb 5, 2021
…le (acasademont)

This PR was merged into the 5.3-dev branch.

Discussion
----------

[PhpUnitBridge] Add SYMFONY_PHPUNIT_REQUIRE env variable

| Q             | A
| ------------- | ---
| Branch?       | 5.x
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | Fix #39387
| License       | MIT
| Doc PR        | symfony/symfony-docs#14913

As discussed on #39387, this PR adds a new `SYMFONY_PHPUNIT_REQUIRE` env variable to add packages to the phpunit bridge installation. This is useful for adding phpunit plugins (ie: https://github.com/phpspec/prophecy-phpunit) without having to add them directly to the main app composer.json.

On my `phpunit.xml.dist` file I can now add

```xml
<server name="SYMFONY_PHPUNIT_REQUIRE" value="phpspec/prophecy-phpunit"/>
```

And the `phpspec/prophecy-phpunit` will be installed along the rest of the phpunit packages

Commits
-------

94e1d87 Add SYMFONY_PHPUNIT_REQUIRE env variable Fixes #39387
symfony-splitter pushed a commit to symfony/phpunit-bridge that referenced this pull request Feb 5, 2021
…le (acasademont)

This PR was merged into the 5.3-dev branch.

Discussion
----------

[PhpUnitBridge] Add SYMFONY_PHPUNIT_REQUIRE env variable

| Q             | A
| ------------- | ---
| Branch?       | 5.x
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | Fix #39387
| License       | MIT
| Doc PR        | symfony/symfony-docs#14913

As discussed on #39387, this PR adds a new `SYMFONY_PHPUNIT_REQUIRE` env variable to add packages to the phpunit bridge installation. This is useful for adding phpunit plugins (ie: https://github.com/phpspec/prophecy-phpunit) without having to add them directly to the main app composer.json.

On my `phpunit.xml.dist` file I can now add

```xml
<server name="SYMFONY_PHPUNIT_REQUIRE" value="phpspec/prophecy-phpunit"/>
```

And the `phpspec/prophecy-phpunit` will be installed along the rest of the phpunit packages

Commits
-------

94e1d877eb Add SYMFONY_PHPUNIT_REQUIRE env variable Fixes #39387
@javiereguiluz javiereguiluz added Testing and removed Waiting Code Merge Docs for features pending to be merged PHPUnitBridge labels Feb 8, 2021
@carsonbot carsonbot changed the title [PHPUnitBridge] Docs for the new SYMFONY_PHPUNIT_REQUIRE env variable [Testing] [PHPUnitBridge] Docs for the new SYMFONY_PHPUNIT_REQUIRE env variable Feb 8, 2021
@javiereguiluz
Copy link
Member

Very nice contribution. Thanks Albert!

@javiereguiluz javiereguiluz merged commit 54e9d51 into symfony:5.x Feb 8, 2021
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.

4 participants