[Php85] Replace null return with empty array in __debugInfo #7124

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

samsonasik merged 2 commits into rectorphp:main from mttsch:feature/php85-__debugInfo-null

Aug 11, 2025

Contributor

mttsch commented Aug 9, 2025

No description provided.

samsonasik reviewed

View reviewed changes

rules/Php85/Rector/ClassMethod/NullDebugInfoReturnRector.php Outdated Show resolved Hide resolved

samsonasik reviewed

View reviewed changes

rules/Php85/Rector/ClassMethod/NullDebugInfoReturnRector.php Outdated Show resolved Hide resolved

mttsch force-pushed the feature/php85-__debugInfo-null branch from 6ff97cf to d9c40c7 Compare

August 10, 2025 11:08

samsonasik reviewed

View reviewed changes

rules/Php85/Rector/ClassMethod/NullDebugInfoReturnRector.php Show resolved Hide resolved

mttsch force-pushed the feature/php85-__debugInfo-null branch 3 times, most recently from d0cbbb5 to f0b15f8 Compare

August 10, 2025 11:21

samsonasik reviewed

View reviewed changes

rules-tests/Php85/Rector/MethodCall/NullDebugInfoReturnRector/Fixture/normal_function.php.inc Outdated Show resolved Hide resolved

samsonasik reviewed

View reviewed changes

...tests/Php85/Rector/MethodCall/NullDebugInfoReturnRector/Fixture/implicit_null_return.php.inc Outdated Show resolved Hide resolved

samsonasik reviewed

View reviewed changes

...tests/Php85/Rector/MethodCall/NullDebugInfoReturnRector/Fixture/implicit_null_return.php.inc Outdated Show resolved Hide resolved

samsonasik reviewed

View reviewed changes

...tests/Php85/Rector/MethodCall/NullDebugInfoReturnRector/Fixture/explicit_null_return.php.inc Outdated Show resolved Hide resolved

samsonasik reviewed

View reviewed changes

...tests/Php85/Rector/MethodCall/NullDebugInfoReturnRector/Fixture/explicit_null_return.php.inc Outdated Show resolved Hide resolved

samsonasik reviewed

View reviewed changes

...tests/Php85/Rector/MethodCall/NullDebugInfoReturnRector/Fixture/explicit_null_return.php.inc Outdated Show resolved Hide resolved

mttsch force-pushed the feature/php85-__debugInfo-null branch from f0b15f8 to a08df1c Compare

August 10, 2025 11:32

Contributor Author

mttsch commented Aug 10, 2025

@samsonasik I adjusted every point.

samsonasik reviewed

View reviewed changes

...ests/Php85/Rector/MethodCall/NullDebugInfoReturnRector/Fixture/skip_different_method.php.inc Outdated Show resolved Hide resolved


          [Php85] Replace null return with empty array in __debugInfo

e2cbed5

mttsch force-pushed the feature/php85-__debugInfo-null branch from a08df1c to e2cbed5 Compare

August 10, 2025 11:36

Member

TomasVotruba commented Aug 11, 2025 •

edited

Loading

@mttsch Could you rebase this one?

Ideally keep new commits without force-push, to it's easier to review and see new changes in git history.


          Merge branch 'main' into feature/php85-__debugInfo-null

0060c5a

samsonasik approved these changes

View reviewed changes

samsonasik merged commit 3dd70ad into rectorphp:main

49 checks passed

Member

samsonasik commented Aug 11, 2025

Thank you @mttsch

mttsch deleted the feature/php85-__debugInfo-null branch

August 11, 2025 15:16

Contributor

github-actions bot commented Jan 11, 2026

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

github-actions bot locked as resolved and limited conversation to collaborators

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

Labels

None yet