search

zendframework / zend-code

BSD 3-Clause "New" or "Revised" License
1.68k stars 78 forks source link

Bumped PHP to 7.1 and updated PHPUnit to ^6.2.2 #114

Closed michalbundyra closed 7 years ago

michalbundyra commented 7 years ago
Ocramius commented 7 years ago

We can bump everything to 7.1, as other zf components are also finally doing it

On 3 Jul 2017 14:50, "Michał Bundyra" notifications@github.com wrote:

These version of PHPUnit are currenty supported. We need both versions because PHPUnit 6 supports only PHP7+.

Changed travis configuration to use PHPUnit 5.7 on PHP 5.6.

You can view, comment on, or merge this pull request online at:

https://github.com/zendframework/zend-code/pull/114 Commit Summary

  • Updated PHUnit to 6.2.2 || 5.7.21

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/zendframework/zend-code/pull/114, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJakGGAO2bU4Tz_-XY7AknMGeeTMz2cks5sKON7gaJpZM4OMPrR .

michalbundyra commented 7 years ago

@Ocramius

We can bump everything to 7.1, as other zf components are also finally doing it

Do you want to do it in this PR?

Ocramius commented 7 years ago

Yep

On 3 Jul 2017 15:17, "Michał Bundyra" notifications@github.com wrote:

@Ocramius https://github.com/ocramius

We can bump everything to 7.1, as other zf components are also finally doing it

Do you want to do it in this PR?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/zendframework/zend-code/pull/114#issuecomment-312643079, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJakOp1kavFwvoKpTtshVQ8dAeeWz4bks5sKOnXgaJpZM4OMPrR .

michalbundyra commented 7 years ago

@Ocramius all done

Ocramius commented 7 years ago

:ship:

Thanks @webimpress!