Skip to content

feat(PHP): Allow PHP 8.4 #49145

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
Merged
nickvergessen merged 6 commits into from
Nov 8, 2024
Merged

feat(PHP): Allow PHP 8.4 #49145

nickvergessen merged 6 commits into from
Nov 8, 2024

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Nov 8, 2024 •
edited
Loading

Follow ups for external storages

Checklist

@nickvergessen nickvergessen added 3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update labels Nov 8, 2024
@nickvergessen nickvergessen added this to the Nextcloud 31 milestone Nov 8, 2024
@nickvergessen nickvergessen self-assigned this Nov 8, 2024
@nickvergessen nickvergessen mentioned this pull request Nov 8, 2024
Open
14 tasks
@nickvergessen
feat(PHP): Allow PHP 8.4
dcddfc6 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
ci: Test on PHP 8.4
da4fe3d 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(logger): Fix closure detection when filtering sensitive parameters
8952a4c 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(util): Correctly create Reflection of method for PHP 8.3+
34592df 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
ci: Don't test incompatible external storages on PHP 8.4
73de407 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
ci: Skip currently PHP 8.4 incompatible test
298948c 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen merged commit 78cf312 into master Nov 8, 2024
187 of 189 checks passed
@nickvergessen nickvergessen deleted the ci/noid/php-8.4 branch November 8, 2024 13:44
@sunjam sunjam mentioned this pull request Nov 12, 2024
Closed
@skjnldsv skjnldsv mentioned this pull request Jan 7, 2025
Merged
@skjnldsv
Copy link
Member

skjnldsv commented Mar 27, 2025

Docs update in nextcloud/documentation@89793e8

@skjnldsv skjnldsv removed the pending documentation This pull request needs an associated documentation update label Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blizzz blizzz blizzz approved these changes

@provokateurin provokateurin provokateurin approved these changes

Assignees

@nickvergessen nickvergessen

Labels

3. to review Waiting for reviews

Projects

None yet

Milestone

Nextcloud 31

Development

Successfully merging this pull request may close these issues.

5 participants

@nickvergessen @skjnldsv @blizzz @provokateurin