Skip to content

[2.3] require-dev PHPUnit bridge #13772

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 24, 2015
Merged

Conversation

nicolas-grekas
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

@stof
Copy link
Member

stof commented Feb 24, 2015

@nicolas-grekas this does not work because the phpunit-bridge requires PHP 5.3.9+ (like all Symfony 2.7+ components)

@stof
Copy link
Member

stof commented Feb 24, 2015

it would be great to fix the deps=high tests too btw.

@fabpot
Copy link
Member

fabpot commented Feb 24, 2015

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 8b02796 into symfony:2.3 Feb 24, 2015
fabpot added a commit that referenced this pull request Feb 24, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

[2.3] require-dev PHPUnit bridge

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Commits
-------

8b02796 [2.3] require-dev PHPUnit bridge
@nicolas-grekas nicolas-grekas deleted the phpunit23 branch February 24, 2015 11:37
ostrolucky pushed a commit to ostrolucky/symfony that referenced this pull request Mar 25, 2018
This PR was merged into the 2.3 branch.

Discussion
----------

[2.3] require-dev PHPUnit bridge

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Commits
-------

8b02796 [2.3] require-dev PHPUnit bridge
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