-
-
Notifications
You must be signed in to change notification settings - Fork 299
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
Switching Margo's Go To Definition to use the types package #571
base: master
Are you sure you want to change the base?
Conversation
…ition or find usage
Conflicts: src/gosubli.me/margo/m_doc.go
Thanks, I'll try and have a look this weekend |
need this so much |
hey, boss, when we could update :) |
I need to sort some stuff out first before I decide whether to merge or not. You don't need to wait for that, just git pull from the fork and use it. |
@DisposaBlackSwan how to "git pull from the fork" to use these features now? What exact git command should I run? |
Is this PR going in sometime? This is a huge feature gap. |
Great improvement, just download dlclark's fork and replace the original GoSublime package. Don't forget to remove the compiled original margo binary in User directory too. |
This change allows Go To Definition to search across a wider variety of use cases. It also adds Find Usages (based on the same code). Most of the file changes are for vendoring the new dependencies.