Skip to content
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

Merged
merged 1 commit into from
Nov 16, 2017
Merged

Add supports for Symfony 4 #12

merged 1 commit into from
Nov 16, 2017

Conversation

mcg-web
Copy link
Member

@mcg-web mcg-web commented Nov 16, 2017

No description provided.

@mcg-web mcg-web force-pushed the symfony_4 branch 2 times, most recently from c000f27 to f60fb9f Compare November 16, 2017 08:24
@overblog overblog deleted a comment from coveralls Nov 16, 2017
@mcg-web mcg-web force-pushed the symfony_4 branch 2 times, most recently from 86834ab to 22d3b15 Compare November 16, 2017 08:48
@overblog overblog deleted a comment from coveralls Nov 16, 2017
Copy link
Member

@renatomefi renatomefi left a 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.*
Copy link
Member

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
Copy link
Member

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?

Copy link
Member Author

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

Copy link
Member

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

@overblog overblog deleted a comment from coveralls Nov 16, 2017
@mcg-web mcg-web force-pushed the symfony_4 branch 2 times, most recently from b5f9815 to e91486f Compare November 16, 2017 09:16
@overblog overblog deleted a comment from coveralls Nov 16, 2017
@mcg-web mcg-web merged commit 005b73f into overblog:master Nov 16, 2017
@mcg-web mcg-web deleted the symfony_4 branch November 16, 2017 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants