-
Notifications
You must be signed in to change notification settings - Fork 516
Playground: Implement PhpdocCommentRule #4074
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
Conversation
c033db6
to
c495d3d
Compare
This pull request has been marked as ready for review. |
1c8c478
to
36d28d7
Compare
|
||
public function getNodeType(): string | ||
{ | ||
return Node::class; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd do it only for Node\Stmt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tbh there are so many different node types, that I don't know what the differences are in this case.
adjusted it per request :)
use function str_starts_with; | ||
|
||
/** | ||
* @implements Rule<Node> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not synced with getNodeType
Thank you. Please add it to https://github.com/phpstan/phpstan/blob/2.1.x/playground-runner/playground.neon once it's built. |
as discussed in phpstan/phpstan#13189 (reply in thread)