Skip to content

feat(rename): link compact strings to local variables#159

Merged
AJenbo merged 1 commit into
PHPantom-dev:mainfrom
calebdw:calebdw/push-ruultkyqsrto
Jun 20, 2026
Merged

feat(rename): link compact strings to local variables#159
AJenbo merged 1 commit into
PHPantom-dev:mainfrom
calebdw:calebdw/push-ruultkyqsrto

Conversation

@calebdw

@calebdw calebdw commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Hello!

This allows strings in compact() to be renamed when renaming the associated variable.

Thanks!

@codecov-commenter

codecov-commenter commented Jun 18, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 92.30769% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/symbol_map/extraction.rs 88.23% 4 Missing ⚠️
src/definition/resolve.rs 66.66% 2 Missing ⚠️
src/references/mod.rs 93.33% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@AJenbo

AJenbo commented Jun 20, 2026

Copy link
Copy Markdown
Collaborator

Nice that's pretty relevant for Laravel

Signed-off-by: Anders Jenbo <anders@jenbo.dk>
@AJenbo AJenbo force-pushed the calebdw/push-ruultkyqsrto branch from a79ef47 to 9430210 Compare June 20, 2026 14:11
@AJenbo AJenbo merged commit 60ce94c into PHPantom-dev:main Jun 20, 2026
7 checks passed
@calebdw calebdw deleted the calebdw/push-ruultkyqsrto branch June 20, 2026 23:48
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.

3 participants