-
Notifications
You must be signed in to change notification settings - Fork 182
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
Dependencies upgrade #379
Dependencies upgrade #379
Conversation
8b60f57
to
a677b3c
Compare
I kept Symfony 4.0.* on allowed failure section because some package are not yet compatible, like phpcr-bundle or paratest. |
Oh, phpunit is completly failing with Symfony 4... I'll update travis and composer to make Symfony 4 test mandatory plus an extra test with problematic dependencies. |
a677b3c
to
984fa14
Compare
984fa14
to
e2f75dd
Compare
Lint errors was found. A patch is also available. Please see the report: https://flintci.io/repositories/1364/analyses/902 |
e2f75dd
to
dd3c64b
Compare
Lint errors was found. A patch is also available. Please see the report: https://flintci.io/repositories/1364/analyses/903 |
BTW @lsmith77, can you please merge master on 2.x? Some fixes are needed like public services. Branches are currently diverged: https://github.com/liip/LiipFunctionalTestBundle/branches |
1868ccd
to
826d2be
Compare
Lint errors was found. A patch is also available. Please see the report: https://flintci.io/repositories/1364/analyses/906 |
Lint errors was found. A patch is also available. Please see the report: https://flintci.io/repositories/1364/analyses/907 |
826d2be
to
cd3ed18
Compare
@soullivaneuh I merged services in #376. |
2977278
to
353525c
Compare
I'm afraid we have to drop LTS support this time because of strict type inheritance: https://travis-ci.org/liip/LiipFunctionalTestBundle/jobs/328587754#L662 |
I just tested this branch on my project, it works like a charm! 💚 Here is my composer installation:
|
Waow, it looks great, thanks for all the work! |
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.
🎉
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.
I've left some comments, but nothing is a blocker, LGTM! 👍
composer.json
Outdated
"symfony/symfony": "^3.4 || ^4.0", | ||
"symfony/phpunit-bridge": "^3.4 || ^4.0", | ||
"doctrine/orm": "^2.5", | ||
"symfony/console": "^3.4 || ^4.0", |
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.
This is duplicated, since you require symfony/symfony
a few lines up
@@ -10,7 +10,7 @@ | |||
> | |||
|
|||
<php> | |||
<server name="KERNEL_DIR" value="tests/App" /> | |||
<server name="KERNEL_CLASS" value="Liip\FunctionalTestBundle\Tests\App\AppKernel" /> |
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.
👍
* | ||
* @see http://www.whitewashing.de/2012/02/25/symfony2_controller_testing.html |
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.
This is getting lost, but it's probably outdated since it's from 2012...
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.
Yes, this can be removed. (I added this link 2 years ago as a reference but most of the code in the blog post is not necessary anymore)
b5e4d10
to
48ec7c6
Compare
Duplicate dependencies on composer.json removed. |
It seems Travis is going away. 😕 |
The build is here (and green): https://travis-ci.org/liip/LiipFunctionalTestBundle/builds/329147881 Apparently the PR status is broken. |
Maybe @lsmith77 can check on the GitHub settings? BTW since it's green IMHO we can merge it! |
we tightened some security settings .. looking into fixing 3rd party integrations .. |
I closed and reopened the PR in order to have the Travis CI analysis. |
It looks like it's good! 👍 |
Thanks a lot @soullivaneuh for the amazing work! |
@alexislefebvre You are welcome! Hope the 2.0 release will be done soon. :-) |
Closes #268
Closes #297
Closes #321
Closes #337 (replace)
Closes #366 (replace)
Closes #378 (replace)
Closes #353
Because some dependencies are linked together (e.g. PHPUnit with Symfony, Nelmio alice with data-fixtures...), I decided to make a global PR to update them all.
~
and simple|
should be used by convention)symfony/symfony
dependencies to^3.4 || ^4.0
when tests will be fixed