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

Skip to content

[VarDumper] Test ReflectionCaster excluding verbosity #23476

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
Jul 11, 2017

Conversation

maidmaid
Copy link
Contributor

@maidmaid maidmaid commented Jul 11, 2017

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

Cover better ReflectionCaster in tests thanks to EXCLUDE_* filter which was added in #22588.

@nicolas-grekas
Copy link
Member

please group your PRs if you can...

@maidmaid
Copy link
Contributor Author

ReflectionCaster is the last case of this kind.

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jul 11, 2017
@nicolas-grekas
Copy link
Member

Thank you @maidmaid.

@nicolas-grekas nicolas-grekas merged commit e535089 into symfony:3.4 Jul 11, 2017
nicolas-grekas added a commit that referenced this pull request Jul 11, 2017
…aidmaid)

This PR was merged into the 3.4 branch.

Discussion
----------

[VarDumper] Test ReflectionCaster excluding verbosity

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

Cover better ``ReflectionCaster`` in tests thanks to ``EXCLUDE_*`` filter which was added in #22588.

Commits
-------

e535089 Add exculde verbosity test
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.

3 participants