Skip to content

[CodingStyle] Remove previous node attribute on TernaryConditionVariableAssignmentRector#3540

Merged
TomasVotruba merged 1 commit intomainfrom
cd-prev-ter
Mar 30, 2023
Merged

[CodingStyle] Remove previous node attribute on TernaryConditionVariableAssignmentRector#3540
TomasVotruba merged 1 commit intomainfrom
cd-prev-ter

Conversation

@samsonasik
Copy link
Member

Replace previous with assigned_to attribute on the ternary.

@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba I think it is ready.

@TomasVotruba TomasVotruba merged commit fd2e1a1 into main Mar 30, 2023
@TomasVotruba TomasVotruba deleted the cd-prev-ter branch March 30, 2023 16:37
@TomasVotruba
Copy link
Member

Awesome 👍

/**
* @var string
*/
public const ASSIGNED_TO = 'assigned_to';
Copy link
Member Author

Choose a reason for hiding this comment

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

@TomasVotruba I just re-check that assigned_to attribute came from symplify/phpstan-rules, which is a require-dev, on scoped build, it is not included, so the AssignedToPropertyNodeVisitor need to be copied to rector-src

https://github.com/symplify/phpstan-rules/blob/main/src/NodeVisitor/AssignedToPropertyNodeVisitor.php#L15

Copy link
Member Author

Choose a reason for hiding this comment

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants