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.

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

go-rename in go-mode.el conflicts with go-rename in go-rename.el #442

Open
gutron opened this issue Jul 12, 2024 · 3 comments
Open

go-rename in go-mode.el conflicts with go-rename in go-rename.el #442

gutron opened this issue Jul 12, 2024 · 3 comments

Comments

@gutron
Copy link

gutron commented Jul 12, 2024

go-rename.el: (defun go-rename (new-name &optional force)

go-mode.el: (defun go-rename ()

And since go-rename.el requires go-mode.el, go-rename.el no longer compiles cleanly.

@dominikh
Copy link
Owner

@adonovan are there any plans to deprecate gorename in the same manner as guru?

@adonovan
Copy link
Contributor

adonovan commented Sep 9, 2024

(Sorry for neglecting this.)

Sorry for defining the conflicting symbols. We don't have a specific proposal to deprecate and delete cmd/gorename, but we probably should as its core functionality is accessible through gopls:

$ cat a.go
package main

func main() {
	f()
}

func f() {}

$ gopls rename -write ./a.go:#42 g

$ cat a.go
package main

func main() {
	g()
}

func g() {}

(gopls doesn't have support the symbolic command-line syntax, but I'm not sure I expect most people invoke it by position, from their editor.)

@adonovan
Copy link
Contributor

adonovan commented Sep 9, 2024

Filed:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants