Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Php cs fixer #19121

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 2 commits into from
Jun 21, 2016
Merged

Php cs fixer #19121

merged 2 commits into from
Jun 21, 2016

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Jun 20, 2016

Q A
Branch? 2.7
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? no
Fixed tickets n/a
License MIT
Doc PR n/a

@nicolas-grekas
Copy link
Member

👍

@Oliboy50
Copy link

Oliboy50 commented Jun 20, 2016

Tests pass? no => Tests pass? yes ?

@Tobion
Copy link
Contributor

Tobion commented Jun 20, 2016

👍

@fabpot fabpot merged commit 7cc3ca5 into symfony:2.7 Jun 21, 2016
fabpot added a commit that referenced this pull request Jun 21, 2016
This PR was merged into the 2.7 branch.

Discussion
----------

Php cs fixer

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | no
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

Commits
-------

7cc3ca5 fixed CS
fdef804 tweaked default CS fixer config
fabpot added a commit that referenced this pull request Oct 16, 2016
This PR was merged into the 2.7 branch.

Discussion
----------

PHP CS Fixer: use php_unit_dedicate_assert

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | see CIs
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

appendix for #19121

Commits
-------

b0df3a7 PHP CS Fixer: use php_unit_dedicate_assert
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.

5 participants