-
-
Notifications
You must be signed in to change notification settings - Fork 301
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
Rename symbol feature #114
Comments
marked as FR. I doubt I'd even look into cross-package refactory as it's not something I'm a fan of but locals and globals should be safe |
You could now integrate gorename. It's designed for both manual cli use as well as direct text-editor integration:
|
@DisposaBoy - +1 for adding support for gorename. |
I have been looking for a way to use gorename in an easy way all day, that would be AWESOME! |
Has anyone looked into this one yet? |
@justinfx - I've taken a stab at it, but I'm not sure how to get sublime to see my $PATH--I'd rather not have to hardcode a path to https://github.com/smartystreets/sublime-gorename/blob/master/gorename.py Am I missing something simple? |
That is something that should already be handled by GoSublime https://github.com/DisposaBoy/GoSublime/blob/master/gosubl/gs.py The env is a settings value that allows you to point it at the desired bin Also you may want to switch your subprocess call to use the list form, and On Sun, 16 Nov 2014 12:58 PM Michael Whatcott notifications@github.com
|
Ok, thanks for the pointers @justinfx. Not sure if I'll be getting to this anytime soon. If my code helps someone else move this forward that's great too. |
Maybe I will take what you started and propose a merge request to GoSublime On Sun, 16 Nov 2014 7:16 PM Michael Whatcott notifications@github.com
|
Go for it! |
+1 @justinfx |
Comments welcome, on merge request #569 |
* dlclark/master: Reduce Margo goto logging in normal mode. Clean test file. Fix issue where output would show error even on successful goto definition or find usage Remove unused imports Added find usages to command list and default keyboard mapping Updated for windows paths Updated vendored code for types, gcimporter, and exact Switch Margo to GoType and added Find Usages Removed extraneous "fmt_enabled" check from GsGoRename Fixes DisposaBoy#114 - Integrate GoRename command
would love to see this merged. it's been 7 months since the pull request I had made. |
me too... |
@DisposaBoy - Thanks for creating a great plugin for go development. I've written so much code with this tool! Being able to safely rename a variable across an entire project from within the editor is a powerful capability. This PR would be a great addition to GoSublime. Once you've had the ability to rename any token at will in other languages, programming in go feels clunky. The good news: The go plugin for Intellij already supports this behavior and it's awesome. Just sayin'. |
Pretty sure the project is abandoned. What with this pull request being a year old. |
A great golang plugin has dead :( |
It would be nice to have a feature to rename symbols (variables, structure fields, functions etc) in a syntax-aware way, and across all files of a package (or even inter-package, but I'm not sure about that).
The text was updated successfully, but these errors were encountered: