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.

Sign up for GitHub

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

Jump to bottom

[2.0] Change minimum php requirement to 7 #270

Open
carbontwelve opened this issue Nov 24, 2017 · 1 comment
Open

[2.0] Change minimum php requirement to 7 #270

carbontwelve opened this issue Nov 24, 2017 · 1 comment
Assignees
@carbontwelve
Labels
add-documentation enhancement in-progress
Milestone
2.0.0

Comments

@carbontwelve
Copy link
Member

carbontwelve commented Nov 24, 2017 •
edited

Support ends on February 2, 2018 for PHPUnit 5; before that happens Tapestry should migrate to begin using PHP 7 and new features that version provides.

This will be version 2 of Tapestry, with version 1 remaining maintained through to 2019, but only receiving patches to any security/serious bugs.

As part of this issue all dependencies should be upgraded to the latest version as supported by PHP 7 and any issues this raises should be fixed.

Also drop HHVM.
@carbontwelve carbontwelve added this to the 2.0.0 milestone Nov 24, 2017
@carbontwelve
Copy link
Member Author

This will also require upgrading all packages installed via composer to php7 varieties, e.g symfony components from version 3.4 to version 4.

@carbontwelve carbontwelve changed the title Change minimum php requirement to 7 [2.0] Change minimum php requirement to 7 Feb 22, 2018
carbontwelve added a commit that referenced this issue Feb 22, 2018
@carbontwelve
Updating dependencies as per #270, this breaks everything...
f3eb7bc
carbontwelve added a commit that referenced this issue Feb 22, 2018
@carbontwelve
(#270) Update for working with 7.0
edf0e28
@carbontwelve carbontwelve self-assigned this Feb 22, 2018
carbontwelve added a commit that referenced this issue Feb 22, 2018
@carbontwelve
(#270) Refactored test for 7.0 - passing
201fb00
carbontwelve added a commit that referenced this issue Feb 22, 2018
@carbontwelve
(#270) Enabling a handfull of tests - passing
c5079c3
carbontwelve added a commit that referenced this issue Feb 22, 2018
@carbontwelve
(#270) lock file was generated on 7.2, needs removing so 7.0 unit tes…
ddf8635 
…ts "pass"
carbontwelve added a commit that referenced this issue Feb 23, 2018
@carbontwelve
Various 2.0 Amends (#304)
13e4464 
* 🚚 added two new steps for #297 in new folder as per #282

* 🚚 drafting things out

* 🔥 disabling 1.* tests for refactoring

* 🚚 tinkering

* 🚚 filling in ProjectFile methods from File

* Refactoring to use ProjectFile

* Updating dependencies as per #270, this breaks everything...

* (#270) Update for working with 7.0

* Store uid in meta array so File object can be duplicated correctly

* (#284) Refactoring project to use ProjectFile

* (#300) Use Plates v4, this restricts us from launching 2.0 until v4 is stable

* (#284) Refactored out File entity usage for ProjectFile

* (#270) Refactored test for 7.0 - passing

* (#270) Enabling a handfull of tests - passing

* (#300) Refactored most Plates Extensions

* (#282) Refactored LoadSourceFileTree

* (#300) Refactored to use Plates v4

* Apply fixes from StyleCI (#301)

Unit tests allowed to fail

* (#270) lock file was generated on 7.2, needs removing so 7.0 unit tests "pass"
carbontwelve added a commit that referenced this issue Oct 11, 2018
@carbontwelve
Changing minimum version to 7.1 due to use of nullable return types #270
f5377e5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carbontwelve carbontwelve

Labels
add-documentation enhancement in-progress
Projects
None yet
Milestone
2.0.0
Development

No branches or pull requests
1 participant
@carbontwelve