-
Notifications
You must be signed in to change notification settings - Fork 416
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
Support #line directive in find references #1660
Conversation
@mholo65 Cake tests fail with this, should I exclude Cake from this span mapping logic? |
@filipw let me have a look at the Cake, and see if I can fix it. Cake, behind the scenes uses a lot of |
did you get a chance to look at this? |
I'm looking now. |
Exclude Cake files from line span mapping
thanks @mholo65 should be good to go |
Fixes #1649
If the mapping goes to an external file, we assume column 0,0