Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Go to definition doesn't work on a #include with an absolute path #13103

Open
sean-mcmanus opened this issue Jan 3, 2025 · 1 comment
Open

Go to definition doesn't work on a #include with an absolute path #13103

sean-mcmanus opened this issue Jan 3, 2025 · 1 comment
Assignees
@sean-mcmanus
Labels
bug Language Service quick fix regression A bug that didn't exist in a previous release Works in VS So we'd need to fix it for VS Code to reach parity.
Milestone
1.23

Comments

@sean-mcmanus
Copy link
Contributor

It works in VS. I think it's a regression.

Originally reported by the user at #13100 .
@sean-mcmanus sean-mcmanus added bug Language Service regression A bug that didn't exist in a previous release labels Jan 3, 2025
@sean-mcmanus sean-mcmanus self-assigned this Jan 3, 2025
@sean-mcmanus sean-mcmanus added this to the 1.23 milestone Jan 3, 2025
@sean-mcmanus sean-mcmanus moved this to In progress in cpptools Jan 3, 2025
@sean-mcmanus sean-mcmanus added the Works in VS So we'd need to fix it for VS Code to reach parity. label Jan 3, 2025
@sean-mcmanus sean-mcmanus mentioned this issue Jan 3, 2025
Open
@sean-mcmanus
Copy link
Contributor Author

It works with 1.19.9 and fails starting with 1.20.5 (or some insider release of 1.20.x).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sean-mcmanus sean-mcmanus

Labels
bug Language Service quick fix regression A bug that didn't exist in a previous release Works in VS So we'd need to fix it for VS Code to reach parity.
Projects
cpptools
Status: In progress
Milestone
1.23
Development

No branches or pull requests
1 participant
@sean-mcmanus