Skip to content

Conversation

@julienfalque
Copy link
Contributor

Q A
Branch? 3.4
Bug fix? no
New feature? yes-ish
Deprecations? no
Tickets symfony/recipes#652 (comment)
License MIT
Doc PR -

In symfony/recipes#652 I would like to make Flex create the phpunit.xml.dist file with a local URI for the phpunit.xsd. This is doable when using phpunit/phpunit standalone because the path to the XSD file is known. This PR aims to allow doing this when using the PHPUnit Bridge, which installs PHPUnit in a path that might change.

Is is simple @copy() call ok? Should I add some error handling?

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.

LGTM thanks :)

@fabpot
Copy link
Member

fabpot commented Sep 30, 2019

Thank you @julienfalque.

fabpot added a commit that referenced this pull request Sep 30, 2019
This PR was merged into the 3.4 branch.

Discussion
----------

Copy phpunit.xsd to a predictable path

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | yes-ish
| Deprecations? | no
| Tickets       | symfony/recipes#652 (comment)
| License       | MIT
| Doc PR        | -

In symfony/recipes#652 I would like to make Flex create the `phpunit.xml.dist` file with a local URI for the `phpunit.xsd`. This is doable when using `phpunit/phpunit` standalone because the path to the XSD file is known. This PR aims to allow doing this when using the PHPUnit Bridge, which installs PHPUnit in a path that might change.

Is is simple `@copy()` call ok? Should I add some error handling?

Commits
-------

233dcb4 Copy phpunit.xsd to a predictable path
@fabpot fabpot merged commit 233dcb4 into symfony:3.4 Sep 30, 2019
@julienfalque julienfalque deleted the local-phpunit-xsd branch October 1, 2019 06:01
This was referenced Oct 7, 2019
@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Oct 26, 2019
@fabpot fabpot mentioned this pull request Nov 12, 2019
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