8000 [3.0] [VarDumper] Tests fixed by TomasVotruba · Pull Request #16396 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[3.0] [VarDumper] Tests fixed #16396

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
Oct 31, 2015
Merged

[3.0] [VarDumper] Tests fixed #16396

merged 1 commit into from
Oct 31, 2015

Conversation

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

Fixes broken test after merge.
Original 2.8 commit: 303f05b#diff-c45f29f2b7f10a6e3583f92526da067dR55

Fixes Travis failure of VarDumper: https://travis-ci.org/symfony/symfony/jobs/88382131#L2413

@Tobion
Copy link
Contributor
Tobion commented Oct 31, 2015

Thank you @TomasVotruba.

@Tobion Tobion merged commit 358fb2d into symfony:master Oct 31, 2015
Tobion added a commit that referenced this pull request Oct 31, 2015
This PR was merged into the 3.0-dev branch.

Discussion
----------

[3.0] [VarDumper] Tests fixed

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

Fixes broken test after merge.
Original 2.8 commit: 303f05b#diff-c45f29f2b7f10a6e3583f92526da067dR55

Fixes Travis failure of VarDumper: https://travis-ci.org/symfony/symfony/jobs/88382131#L2413

Commits
-------

358fb2d HtmlDumperTest fixed after merge
@TomasVotruba
Copy link
Contributor Author

👍

@TomasVotruba TomasVotruba deleted the test-fixes branch October 31, 2015 03:53
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
0