-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[PHPUnit bridge] Bump php version of PHPUnit-bridge #29718
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
Conversation
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.
while bumping to phpunit5 & php5.5, there are many opportunities to improve the code (eg $this in closure, and polyfills that we could remove, see "Legacy" namespace.)
Up to do it?
haven't bumped PHPUnit version..
it's not that I don't want to help.. but I am not sure about which changes and what you are thinking exactly.. so we might have a lot going back and forth discussion.. I suggest to have these changes on other PRs.. |
phpunit 5 is for php56 😞
works for me! |
@@ -5,6 +5,7 @@ CHANGELOG | |||
----- | |||
|
|||
* added `ClassExistsMock` | |||
* bumped PHP version from 5.5.3 to 5.5.9 |
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.
Is it a typo? 5.3.3
?
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 it is a type.. fixed.
The |
e4f7a09
to
5931309
Compare
Thank you @gmponos. |
…mponos) This PR was squashed before being merged into the 4.3-dev branch (closes #29718). Discussion ---------- [PHPUnit bridge] Bump php version of PHPUnit-bridge | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no (?) | Deprecations? | no | Tests pass? | yes | License | MIT Related to #29623 (comment) I checked the BC page I think I couldn't find any related to BC changes... anyway... As far as I understood from the comment above I must target `master` branch. Commits ------- 5931309 [PHPUnit bridge] Bump php version of PHPUnit-bridge
…las-grekas) This PR was merged into the 4.3-dev branch. Discussion ---------- [PHPUnitBridge] Static access in closure | Q | A | ------------- | --- | Branch? | master for features | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n / a | License | MIT | Doc PR | n / a #29718 bumps php to 5.5 . This is what we could simplify if it gets merged, thanks to this "new" PHP feature: https://3v4l.org/sJOWr EDIT: it was merged, this can be reviewed. Commits ------- 97f3023 Use the scope of an instance 9df76eb Extract closures into static functions
… (greg0ire) This PR was merged into the 4.3-dev branch. Discussion ---------- [PhpUnitBridge] Url encoded deprecations helper config | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | yes | Tests pass? | yes | Fixed tickets | #28048 | License | MIT | Doc PR | symfony/symfony-docs#10701 First stab at implementing a new way of configuring the deprecation error handler. Includes a refactoring to keep things manageable. Rework of #24867, blocked by #29718 TODO: - [x] make the code 5.5 compatible 😢 - [x] add more tests - [x] deprecate modes (using echo :P) - [x] test this on real life projects and add some screenshots - [x] docs PR - [x] handle `strict` - [x] adapt existing CI config # Quiet configuration  # Default configuration  Commits ------- 1c73f9c [PhpUnitBridge] Url encoded deprecations helper config
Related to #29623 (comment)
I checked the BC page I think I couldn't find any related to BC changes... anyway...
As far as I understood from the comment above I must target
master
branch.