Skip to content

[CC-3237] Add php 8.5 to composer.json#224

Merged
Ryouzanpaku merged 5 commits intomainfrom
CC-3237/php_8_5_support
Feb 19, 2026
Merged

[CC-3237] Add php 8.5 to composer.json#224
Ryouzanpaku merged 5 commits intomainfrom
CC-3237/php_8_5_support

Conversation

@Ryouzanpaku
Copy link
Copy Markdown
Contributor

@Ryouzanpaku Ryouzanpaku commented Feb 17, 2026

Composer and PHPUnit configuration:

  • Updated composer.json to require PHP 8.1+ and PHPUnit 9.6+
  • Dropping support for PHP 7.4 and 8.0.

Cleanup deprecations

  • Enabled conversion of deprecations to exceptions in phpunit.xml for stricter test enforcement.
  • removed usage of deprecated method, curl_close

Test reporting

  • Adding test reports to test workflows.

@Ryouzanpaku Ryouzanpaku requested a review from a team as a code owner February 17, 2026 13:55
@Ryouzanpaku Ryouzanpaku force-pushed the CC-3237/php_8_5_support branch from 8bc304d to 3379ebc Compare February 18, 2026 12:01
@Ryouzanpaku Ryouzanpaku force-pushed the CC-3237/php_8_5_support branch from 3379ebc to ed9eed4 Compare February 18, 2026 12:06
- `curl_close`
- `\ReflectionMethod::setAccessible`
@Ryouzanpaku Ryouzanpaku force-pushed the CC-3237/php_8_5_support branch from 94fb9ac to 524ab31 Compare February 18, 2026 12:51
@Ryouzanpaku Ryouzanpaku force-pushed the CC-3237/php_8_5_support branch from 524ab31 to 2d76d2b Compare February 18, 2026 13:44
*/
public function close(): void
{
curl_close($this->request);
Copy link
Copy Markdown
Contributor Author

@Ryouzanpaku Ryouzanpaku Feb 18, 2026

Choose a reason for hiding this comment

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

curl_close got removed because it is deprecated and did not do anything from php 8.0 onwards.

@Ryouzanpaku Ryouzanpaku merged commit 8c28ade into main Feb 19, 2026
13 of 14 checks passed
@Ryouzanpaku Ryouzanpaku deleted the CC-3237/php_8_5_support branch February 19, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants