Skip to content

Detect dead properties#277

Merged
janedbal merged 19 commits intomasterfrom
dead-properties
Dec 18, 2025
Merged

Detect dead properties#277
janedbal merged 19 commits intomasterfrom
dead-properties

Conversation

@janedbal
Copy link
Copy Markdown
Member

@janedbal janedbal commented Dec 5, 2025

Closes #252

@janedbal janedbal force-pushed the dead-properties branch 3 times, most recently from b5694cc to ae8cb72 Compare December 15, 2025 11:47
@janedbal janedbal marked this pull request as ready for review December 18, 2025 12:47
@janedbal janedbal merged commit 08c3975 into master Dec 18, 2025
66 of 68 checks passed
@janedbal janedbal deleted the dead-properties branch December 18, 2025 16:44
@ruudk
Copy link
Copy Markdown
Contributor

ruudk commented Dec 18, 2025

Epic! Will this be tagged? Thanks so much!!

@janedbal
Copy link
Copy Markdown
Member Author

Not yet, this MR is very simple towards properties. Truely advanced analysis is almost-ready in #281, I'm currently testing it on multiple repositories and composing some yet-to-decide edgecases. We will support even neverRead and neverWritten parts and all that combined with property hook craziness and transitivity. All while supporting even writes via promoted properties or ASTs like [$foo->first, $foo->last] = explode() or read-writes like return $this->foo ??= 1; etc

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.

Detect unused public properties

2 participants