Skip to content

"Find All References" For Required File #14780

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

Closed
mjbvz opened this issue Mar 22, 2017 · 0 comments · Fixed by #15737
Closed

"Find All References" For Required File #14780

mjbvz opened this issue Mar 22, 2017 · 0 comments · Fixed by #15737
Labels
Fixed A PR has been merged for this issue Suggestion An idea for TypeScript VS Code Tracked There is a VS Code equivalent to this issue

Comments

@mjbvz
Copy link
Contributor

mjbvz commented Mar 22, 2017

From: microsoft/vscode#22977

TypeScript Version: 2.2.1
Code

a.ts

export const abc = 123

b.ts

import {abc} from './a'

Send a references command to the TSServer for the ./a in b.ts

Expected behavior:
Returns results for all places where a.ts is imported or required

Actual behavior:
No results returned

@mjbvz mjbvz added the VS Code Tracked There is a VS Code equivalent to this issue label Mar 22, 2017
@mhegazy mhegazy added the Suggestion An idea for TypeScript label Mar 23, 2017
@mhegazy mhegazy added this to the Future milestone Mar 23, 2017
@mhegazy mhegazy assigned ghost Mar 23, 2017
@mhegazy mhegazy added the Fixed A PR has been merged for this issue label May 18, 2017
@mhegazy mhegazy modified the milestones: TypeScript 2.4, Future May 18, 2017
@ghost ghost closed this as completed in #15737 May 19, 2017
@microsoft microsoft locked and limited conversation to collaborators Jun 21, 2018
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Fixed A PR has been merged for this issue Suggestion An idea for TypeScript VS Code Tracked There is a VS Code equivalent to this issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants