Skip to content

[Php81] Remove usage of AttributeGroupNewLiner on ReadOnlyPropertyRector#7535

Merged
samsonasik merged 6 commits into
mainfrom
remove-on-readonly-prop
Oct 22, 2025
Merged

[Php81] Remove usage of AttributeGroupNewLiner on ReadOnlyPropertyRector#7535
samsonasik merged 6 commits into
mainfrom
remove-on-readonly-prop

Conversation

@samsonasik
Copy link
Copy Markdown
Member

@TomasVotruba per your request at

#7531 (comment)

here remove unused AttributeGroupNewLiner service on property on ReadOnlyPropertyRector.

The existing fixture already inline, so keep inline as is, and add another fixture for new line, to add new line.

@samsonasik samsonasik enabled auto-merge (squash) October 22, 2025 11:46
@samsonasik samsonasik merged commit c0599cd into main Oct 22, 2025
52 checks passed
@samsonasik samsonasik deleted the remove-on-readonly-prop branch October 22, 2025 11:47
{
#[Serializer\Since(Option::SINCE_20211124)]
private readonly string $name;
#[Serializer\Since(Option::SINCE_20211124)]private readonly string $name;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@github-actions
Copy link
Copy Markdown
Contributor

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 github-actions Bot locked as resolved and limited conversation to collaborators Mar 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants