Abdul Malik Ikhsan
Abdul Malik Ikhsan
@TomasVotruba mergeable?
@TomasVotruba ready 👍
rector-symfony CI already fixed at latest main branch :) - https://github.com/rectorphp/rector-src/pull/7772 - https://github.com/rectorphp/rector-symfony/pull/911
All checks have passed 🎉 @TomasVotruba it is ready for review.
I tested in our project with 5617 files with 101 files changed: _Before_: **_1 minutes 52 seconds_** ```bash ➜ ../rector-src/bin/rector process --clear-cache [OK] 101 files have been changed by Rector...
@peterfox unit test somehow error
The rule purpose is only remove `@return` tag when there is native return type. For removing `@return` tag when there is no native type, that should be in separate rule...
Add another rule under rules/DeadCode/Rector/ClasaMethod, add test under rules-tests/DeadCode/Rector/ClassMethod as well
No need, just new rule under DeadCode\Recror\ClassMethod is fine imo :)
Tested as well on rector- packages: - rector-phpunit - rector-downgrade-php with require-dev: ```json "phpstan/phpstan": "2.1.x-dev", "rector/rector-src": "dev-test-phpstan-21x", ``` and seems working ok 👍