pkp / pkp-lib

The library used by PKP's applications OJS, OMP and OPS, open source software for scholarly publishing.

Home Page:https://pkp.sfu.ca

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Upgrade to Laravel version 11.x

touhidurabir opened this issue · comments

Describe the Proposal
With the intension to make the release of 3.5.0 a LTS, we should aim to have all the dependencies updated as much as possible . The current main(pre release of 3.5.0) now have Laravel 10.x as the dependency which will be get bug fix till August 6th, 2024 . Also by the time the 3.5.0 get to release it will be an older version of laravel . Better to upgrade the laravel version to 11.x .

Additional Information
With the release of stable laravel version 11.x at March 12th, 2024 which seems like a more of better candidate for next LTS release as it will have a longer support period, more details at https://laravel.com/docs/11.x/releases#support-policy . The most important requirement for it is PHP 8.2+ which is also seems like a good opportunity to ship the next LTS with more actively supported PHP version as PHP 8.1 will be at the end of active support by next release, more details at https://www.php.net/supported-versions.php .

The upgrade process and details does not seems to contains many high impact changes and most of those seems to be resolvable with less effort, more details at https://laravel.com/docs/11.x/upgrade .

PRs
pkp-lib --> #10211
ojs --> pkp/ojs#4367

@touhidurabir, I checked with the Technical Committee, and they didn't raise any problems with bumping our PHP base to 8.2 for OJS/OMP/OPS 3.5. They're primarily familiar with RHEL (EPEL) and Debian/Ubuntu.

@asmecher can you review it at #9892 (comment) . Unit tests are still failing as we need to update the github workflow main branch to make it phpunit test command compatible with version 11 .

@touhidurabir, I think you're probably referring to this change, which got merged last week; did it resolve the problem?

I've added reviews to ojs and pkp-lib, thanks, @touhidurabir!

@touhidurabir, I think you're probably referring to this change, which got merged last week; did it resolve the problem?

@asmecher that was one part but I missed another one where set flag -v not supported by PHPUnit 11 anymore . I will create a new PR soon for this .