Skip to content

[CodeQuality] Skip @final doc with public property on ConvertStaticToSelfRector#7168

Merged
samsonasik merged 2 commits intomainfrom
skip-final-doc-prop
Aug 23, 2025
Merged

[CodeQuality] Skip @final doc with public property on ConvertStaticToSelfRector#7168
samsonasik merged 2 commits intomainfrom
skip-final-doc-prop

Conversation

@samsonasik
Copy link
Member

@calebdw I got regression.

Final via @final doc should be skipped on public property. This PR fix it.

@samsonasik samsonasik force-pushed the skip-final-doc-prop branch from 8c66099 to d636aec Compare August 23, 2025 18:59
@samsonasik
Copy link
Member Author

All checks have passed 🎉 @TomasVotruba I am merging it ;)

@samsonasik samsonasik merged commit e95966d into main Aug 23, 2025
49 checks passed
@samsonasik samsonasik deleted the skip-final-doc-prop branch August 23, 2025 19:08
@github-actions
Copy link
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 Jan 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant