Skip to content

Conversation

tomasfejfar
Copy link
Contributor

Followup to #1080

@tomasfejfar tomasfejfar force-pushed the tf-locateIdentifiersByType branch 3 times, most recently from 3ec23e3 to 08c63d9 Compare March 25, 2022 12:34
Copy link
Member

@ondrejmirtes ondrejmirtes left a comment

Choose a reason for hiding this comment

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

Why not constants too? 😊

@tomasfejfar
Copy link
Contributor Author

@ondrejmirtes I kind of extrapolated from #1080 (comment). It felt weird that if I'm looking in directory constants are not located, but when I look at single file, they are.

@ondrejmirtes
Copy link
Member

I just want locateIdentifier() and locateIdentifiersByType() to be consistent in each locator.

@ondrejmirtes ondrejmirtes changed the base branch from 1.5.x to 1.9.x October 26, 2022 09:02
@ondrejmirtes ondrejmirtes force-pushed the tf-locateIdentifiersByType branch from 08c63d9 to 54c41bc Compare October 26, 2022 09:14
@ondrejmirtes
Copy link
Member

I took it over the finish line, thanks.

@ondrejmirtes ondrejmirtes merged commit 9914301 into phpstan:1.9.x Oct 26, 2022
@tomasfejfar tomasfejfar deleted the tf-locateIdentifiersByType branch October 26, 2022 19:39
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