Skip to content

IDE: Support textDocument/implementation #5401

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

Merged
merged 2 commits into from
Nov 13, 2018

Conversation

Duhemm
Copy link
Contributor

@Duhemm Duhemm commented Nov 7, 2018

Works with ⌘F12 / <ctrl>F12, or Go > Go to Implementation

Depends on #5209, only the last commit is new.

@Duhemm Duhemm added the area:ide label Nov 7, 2018
@Duhemm Duhemm force-pushed the topic/ide-implementation branch from 8477cd4 to 74ad15c Compare November 7, 2018 17:23
Copy link
Member

@smarter smarter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last commit LGTM

This is used to find implementations of the symbol under the cursor.
@Duhemm Duhemm force-pushed the topic/ide-implementation branch from 74ad15c to 858731b Compare November 13, 2018 07:12
@Duhemm
Copy link
Contributor Author

Duhemm commented Nov 13, 2018

@smarter I've addressed your comments, do you want to take another look?

@Duhemm Duhemm merged commit bdd932f into scala:master Nov 13, 2018
@Duhemm Duhemm deleted the topic/ide-implementation branch November 13, 2018 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants