-
Notifications
You must be signed in to change notification settings - Fork 770
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix go-to-definition to symbols in large files #13987
Conversation
@dotnet/fsharp-team-msft The fix is so simple and obviously correct, and constructing a large test case would take some time, I think we should do that in a separate PR. |
@vzarytovskii Probably worth getting into 17.4 |
Yes, we should cherry-pick it. |
It also fixes #6583. 🙂 |
It actually should probably target 17.4 branch directly. |
@vzarytovskii retargeted to 17.4, can you approve please? Thanks Is 17.4 flowing to main at the moment? |
@vzarytovskii actually will need to cherry pick it separately I think, I'll create a separate PR for that |
@auduchinok I should have read that issue more carefully, sorry |
This is a good fix |
Such a simple fix...
The problem kicks whenever we're going to symbols in files that have above 10000 symbol resolutions
Fixes #13986