Closed
Conversation
Contributor
Author
|
Passed - good. This will need to be planned for some future release. |
Contributor
|
Docs relevant when going to be merged. |
1c7a249 to
819851b
Compare
Contributor
|
💥 Acceptance tests pipeline apiProxySmoke-8-8-mariadb10.2-php7.4 failed. The build has been cancelled. |
Contributor
|
💥 Acceptance tests pipeline webUICreateDelete-chrome-mariadb10.2-php7.4 failed. The build has been cancelled. |
819851b to
a0d00a5
Compare
a0d00a5 to
3a08e76
Compare
|
Kudos, SonarCloud Quality Gate passed! |
Contributor
Author
|
See #40248 for an updated PR |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.








Description
This is a PoC demonstrating the changes needed to drop PHP 7.3
Note: because of issue #40103 I cannot do
composer updatein this PR, or in core master. So the PHP dependencies may be a bit out-of-date here. After that issue is resolved (for example, by merging the guzzle v7 PR), then I can rebase this and update the dependencies.composer found the following dependencies that it could update:
apps/files_external/3rdpartyis also updated.Other things that could be done are:
composer.jsonand see what could be updatedRelated Issue
How Has This Been Tested?
CI
Types of changes
Checklist:
The previous PR was #40005