Skip to content

IBX-11179: Updated PHP versions in CI configuration to 8.3 and 8.4#1823

Open
wiewiurdp wants to merge 3 commits into4.6from
IBX-11179-PHP-8.4-certification-4.6
Open

IBX-11179: Updated PHP versions in CI configuration to 8.3 and 8.4#1823
wiewiurdp wants to merge 3 commits into4.6from
IBX-11179-PHP-8.4-certification-4.6

Conversation

@wiewiurdp
Copy link
Contributor

@wiewiurdp wiewiurdp commented Feb 3, 2026

🎫 Issue IBX-11179

Description:

Updated PHP versions in CI configuration to 8.3 and 8.4, aligning baseline files and removing phpstan errors

@wiewiurdp wiewiurdp force-pushed the IBX-11179-PHP-8.4-certification-4.6 branch from b9fcfb2 to c0a1da7 Compare February 3, 2026 13:45
@wiewiurdp wiewiurdp marked this pull request as ready for review February 4, 2026 08:05
@wiewiurdp wiewiurdp requested a review from a team February 4, 2026 08:06
@ibexa-workflow-automation-1 ibexa-workflow-automation-1 bot requested review from Steveb-p, ViniTou, alongosz, barw4, ciastektk, konradoboza, mikadamczyk and tbialcz and removed request for a team February 4, 2026 08:06
$this->expectExceptionMessage('Expected a ' . ContentInfo::class . ' object.');

$this->contentInfoTransformer->transform($value);
$result = $this->contentInfoTransformer->transform($value);
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this change?


/** @phpstan-ignore method.resultUnused */
$transformer->transform($value);
$result = $transformer->transform($value);
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's keep the previous one, assigning this to variable doesn't make much sense. Besides, we agreed that @phpstan-ignore can be accepted in the tests' related code.

@wiewiurdp wiewiurdp force-pushed the IBX-11179-PHP-8.4-certification-4.6 branch from b06edb2 to 21d66d9 Compare February 5, 2026 13:26
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 5, 2026

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants