-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add supports for Symfony 4 #12
Conversation
c000f27
to
f60fb9f
Compare
86834ab
to
22d3b15
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks nice, I have a couple of comments!
@@ -15,27 +19,35 @@ matrix: | |||
- php: 5.5 | |||
env: SYMFONY_VERSION=2.8.* | |||
- php: 5.6 | |||
env: SYMFONY_VERSION=3.0.* | |||
- php: 7.0 | |||
env: SYMFONY_VERSION=3.1.* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe keep one 7.0 build?
.travis.yml
Outdated
env: SYMFONY_VERSION=3.2.* | ||
- php: hhvm | ||
env: PHP_CS_FIXER=true | ||
- php: 7.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about adding prefer-lowest here as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prefer-lowest fails because of phpunit 4, I'll added in a next PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the problem is only the TestCase class you can set the minimum phpunit to version 4.8.35
https://github.com/sebastianbergmann/phpunit/blob/master/ChangeLog-4.8.md
b5f9815
to
e91486f
Compare
No description provided.