You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
You are right, this is not handled well, but the warning has a reason: It tells you that refactoring such as renaming will most likely not do the correct thing. You can turn the whole inspection off if you like, which was the default behavior of the last major version.
It all boils down to the same reason I described in your last issue: I cannot make it correct without being able to evaluate code on the fly which will, for various reason, never be possible.
Just imagine you used MyPack`Private`one inside MyOtherPackage in my last example
To know that it is the same symbol as the one above, I needed to be able to know what includeMyFile is doing and this is impossible without a real evaluator. This is only one trivial example of things I've seen in the wild.
However, I planned to work on improving the situation especially for the new/alternative way of writing packages. When you read carefully through the post and the references (especially, the answer from Leonid), you'll find that it improves the readability of code. I'd like to leave this issue open because your issue will work correctly after improving some of the algorithms.
Describe the bug
When the full context is used to refer to a symbol, the unresolved symbol inspection fails.
To Reproduce
A minimal example is show in the screenshot below.
Expected behaviour
Fully scoped symbols should be recognized. If this is not viable I would expect them to be ignored by this inspection.
Screenshots

Please also provide:
The text was updated successfully, but these errors were encountered: