Abdul Malik Ikhsan
Abdul Malik Ikhsan
The issue seems because of using `:` with `endif` syntax, when changed to `{ }`, it is working as expected.
@TomasVotruba I tried to apply on `PHPStanNodeScopeResolver`: https://github.com/rectorphp/rector-src/blob/e9e2877d959f24ee91e5f76a6cf626499011c4bb/packages/NodeTypeResolver/PHPStan/Scope/PHPStanNodeScopeResolver.php#L136 and it seems the `$nodes` still uses old statements, even on `FileProcessor`: https://github.com/rectorphp/rector-src/blob/347b5938475e5b0e846be6ef631d5939a47fc1a0/src/Application/FileProcessor.php#L33 It seems we need to get the new statement...
Could you create simple reproducible repository that cause it? Thank you.
We need to have reproducible code that crash without the patch, you can narrow half by half until found a rule that caused it, `--debug` should help
For very large codebase, my suggestion is to split into multiple commands: ``` vendor/bin/rector process src/Controller vendor/bin/rector process src/Model etc ``` that should handle and help narrow which part that...
I can't reproduce, could you create minimal reproduced in demo page https://getrector.org/demo ?
@TomasVotruba thank you, I pinned this to ease to locate
throw Exception should return `never` type, there is `ReturnNeverTypeRector` for that, ref https://getrector.org/demo/e9980211-c436-408e-8c5e-d7a18452364c
`AddVoidReturnTypeWhereNoReturnRector` is using `SilentVoidResolver` service, which check against `never` type as well, which if found, it just skipped https://github.com/rectorphp/rector-src/blob/5ec4cff203f68d9578719b54ce3a145efb32bc79/rules/TypeDeclaration/TypeInferer/SilentVoidResolver.php#L37 I think skip is valid to avoid cross rules apply invalid...
That's seems parallel process, could you try with disable parallel: ```php $rectorConfig->disableParallel(); ``` and see if it is working? If you're running on windows, I don't have windows11 at my...