Skip to content

PHP 8.4 compatibility#209

Merged
baywet merged 2 commits into
microsoftgraph:mainfrom
hubipe:main
Mar 28, 2025
Merged

PHP 8.4 compatibility#209
baywet merged 2 commits into
microsoftgraph:mainfrom
hubipe:main

Conversation

@hubipe

@hubipe hubipe commented Mar 28, 2025

Copy link
Copy Markdown
Contributor

Method signatures updated to be compatible with PHP 8.4

@hubipe
hubipe requested a review from a team March 28, 2025 08:28
@hubipe

hubipe commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@rkodev

rkodev commented Mar 28, 2025

Copy link
Copy Markdown

@hubipe You can add php v8.4 the test version matrix Also please refer to this comment in order to ensure compatibility with version 8.4

@hubipe hubipe mentioned this pull request Mar 28, 2025
@hubipe

hubipe commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

@rkodev Added. Issue #210 filed.

@baywet baywet linked an issue Mar 28, 2025 that may be closed by this pull request

@baywet baywet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution!

@baywet
baywet merged commit fd8cc94 into microsoftgraph:main Mar 28, 2025
@calebdw

calebdw commented Feb 5, 2026

Copy link
Copy Markdown

@baywet, can you please release this? I'm still getting these deprecation errors because 2.4.0 was never tagged and released (packagist still has 2.3.1)

https://packagist.org/packages/microsoft/microsoft-graph-core

@baywet

baywet commented Feb 5, 2026

Copy link
Copy Markdown
Member

I have transitioned to a new team back in August 2025 and won't be replying to this issue/pull request moving forward.
Thank you for your understanding.

@adrian05-ms @gavinbarron should be able to take it from here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for php 8.4

4 participants