Skip to content

Copy phpunit.xsd to a predictable path #33776

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
Sep 30, 2019

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