Skip to content

IBX-9727: Added type-hints and adapted codebase to PHP8+ for Behat-related code #1674

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.

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

Merged
merged 3 commits into from
Aug 13, 2025

Conversation

konradoboza
Copy link
Contributor

@konradoboza konradoboza commented Aug 8, 2025

🎫 Issue IBX-9727

Depends on:

Description:

Regressions: ibexa/commerce#1470

Reworked Behat code to rely on strict typing and PHP8.3 syntax alongside fixing lots of PHPStan issues. Making CI green was possible thanks to invaluable @katarzynazawada assistance. 🥇

For QA:

Documentation:

@konradoboza konradoboza self-assigned this Aug 8, 2025
@konradoboza konradoboza force-pushed the introduced-type-hints-3 branch 6 times, most recently from 4f97508 to 29467be Compare August 11, 2025 12:05
@konradoboza konradoboza force-pushed the introduced-type-hints-2 branch from 87b96df to 23383d9 Compare August 11, 2025 12:42
Base automatically changed from introduced-type-hints-2 to main August 11, 2025 13:12
@konradoboza konradoboza force-pushed the introduced-type-hints-3 branch 2 times, most recently from 65d7f5b to 0dd28e0 Compare August 11, 2025 14:04
@konradoboza konradoboza force-pushed the introduced-type-hints-3 branch 2 times, most recently from 60e9098 to 9ceb454 Compare August 12, 2025 07:46
@konradoboza konradoboza force-pushed the introduced-type-hints-3 branch from 9ceb454 to 2594ec2 Compare August 12, 2025 09:28
Copy link

@konradoboza konradoboza requested a review from a team August 12, 2025 12:31
@ezrobot ezrobot requested review from micszo, pawlakadrian, barbaragr, katarzynazawada and tomaszszopinski and removed request for a team August 12, 2025 12:32
Copy link
Contributor

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

👍 🥇

Copy link
Contributor

@KamilSznajdrowicz KamilSznajdrowicz left a comment

Choose a reason for hiding this comment

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

🦸

Copy link
Contributor

@micszo micszo left a comment

Choose a reason for hiding this comment

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

Thanks @konradoboza & @katarzynazawada! 💪🏻

@micszo micszo merged commit 540f0a1 into main Aug 13, 2025
33 checks passed
@micszo micszo deleted the introduced-type-hints-3 branch August 13, 2025 10:13
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.

8 participants