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

Skip to content

[travis] Build phpunit with local components #15457

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
Aug 5, 2015

Conversation

nicolas-grekas
Copy link
Member

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

The tests for PHP7 currently fail because travis ships with a phar phpunit that embeds the yaml components. This one takes priority over the one in the repo. But it is outdated.
The issue arose with PHP7 tests but could have happen with any build.

@nicolas-grekas
Copy link
Member Author

This PR is the first to have green tests for PHP7!

@nicolas-grekas nicolas-grekas force-pushed the travis-phpunit branch 3 times, most recently from 2bfd734 to 4e70636 Compare August 5, 2015 15:08
@fabpot
Copy link
Member

fabpot commented Aug 5, 2015

Thank you @nicolas-grekas.

@fabpot fabpot merged commit ce2a371 into symfony:2.3 Aug 5, 2015
fabpot added a commit that referenced this pull request Aug 5, 2015
…ekas)

This PR was merged into the 2.3 branch.

Discussion
----------

[travis] Build phpunit with local components

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

The tests for PHP7 currently fail because travis ships with a phar phpunit that embeds the yaml components. This one takes priority over the one in the repo. But it is outdated.
The issue arose with PHP7 tests but could have happen with any build.

Commits
-------

ce2a371 [travis] Build phpunit with local components
@nicolas-grekas nicolas-grekas deleted the travis-phpunit branch August 5, 2015 15:27
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