8000 Drop HHVM support by lcobucci · Pull Request #118 · doctrine/collections · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Drop HHVM support #118

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
May 20, 2017
Merged

Drop HHVM support #118

merged 6 commits into from
May 20, 2017

Conversation

lcobucci
Copy link
Member

Also organising the build and configuration files. Related to doctrine/orm#6424

lcobucci added 6 commits May 20, 2017 16:27
@lcobucci
Run scrutinizer on PHP 7.1
54e62b4
@lcobucci
Remove "performance" group exclusion from PHPUnit config
bca5332 
This group is just not used in doctrine/collections and it doesn't make
sense to keep it here
@lcobucci
Only generate coverage information when it's needed
e8a0bf3 
In order to improve the build execution time.
@lcobucci
Use the provided XSD for PHPunit configuration
5381b95 
Removing unexisting parameters and also some that just had the default
value.
@lcobucci
Disable sudo mode in Travis-CI
c4e30c9
@lcobucci
Drop HHVM support
bf564b6
@lcobucci lcobucci self-assigned this May 20, 2017
@lcobucci lcobucci requested a review from Ocramius May 20, 2017 14:31
@Ocramius Ocramius added this to the 1.5 milestone May 20, 2017
Ocramius
Ocramius approved these changes May 20, 2017
View reviewed changes
Copy link
Member
@Ocramius Ocramius left a comment A50E

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Ocramius Ocramius merged commit 38065c6 into master May 20, 2017
@Ocramius Ocramius deleted the drop-hhvm-support branch May 20, 2017 14:41
@lcobucci lcobucci mentioned this pull request May 20, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ocramius Ocramius Ocramius approved these changes

Assignees

@lcobucci lcobucci

Labels
Enhancement
Projects
None yet
Milestone
1.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@lcobucci @Ocramius
0