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

[book] [testing] fixed the path of the phpunit.xml file #3734

Merged
merged 1 commit into from
Apr 2, 2014
Merged

[book] [testing] fixed the path of the phpunit.xml file #3734

merged 1 commit into from
Apr 2, 2014

Conversation

javiereguiluz
Copy link
Member

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets -

Sometimes is referenced as phpunit.xml and other times as
app/phpunit.xml I think that we should always use the explicit
app/phpunit.xml path, except when explaining the phpunit.xml
file in general.

Sometimes is referenced as phpunit.xml and other times as
app/phpunit.xml I think that we should always use the explicit
app/phpunit.xml path, except when explaining the phpunit.xml
file in general.
@wouterj
Copy link
Member

wouterj commented Mar 29, 2014

👍 This is one of the oldest not updated articles of the documentation. It needs a rewrite sometime.

weaverryan added a commit that referenced this pull request Apr 2, 2014
…viereguiluz)

This PR was merged into the 2.3 branch.

Discussion
----------

[book] [testing] fixed the path of the phpunit.xml file

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3+
| Fixed tickets | -

Sometimes is referenced as phpunit.xml and other times as
app/phpunit.xml I think that we should always use the explicit
app/phpunit.xml path, except when explaining the phpunit.xml
file in general.

Commits
-------

a1f5cd6 [book] [testing] fixed the path of the phpunit.xml file
@weaverryan weaverryan merged commit a1f5cd6 into symfony:2.3 Apr 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants