Change function support for golang

[abhipranay]

Thanks for the plugin. Can you please share steps how I can extend the plugin for golang.
I tried to read the readme but couldn't figure out exactly what is needed to add any new language support.

I tried to add below queries to identify simple functions and their parameters.

I have below simple code. I want to change order of parameters of the function A.

func A(p1 int, p2 string) {
	fmt.Println("hello")
}

func main() {
	A(1, "2")
}

(call_expression
  function: (identifier) @function
  arguments: (argument_list (_) @argument.inner)
)

(function_declaration
  name: (identifier)
  parameters: (parameter_list (_) @parameter.inner)
)

(call_expression
  function: (selector_expression 
              field: (field_identifier) @method
              )
  arguments: (argument_list (_) @argument.inner)
)

I can see the popup and parameters are identified properly.

But when i Confirm in dialog nothing happens.

[SleepySwords]

Thanks for checking out my plugin!

The @argument.inner should be @parameter.inner for now, I may change it later on to accept both so that we can handle arguments and parameters differently in the future, but for now, there is only @parameter.inner.

[abhipranay]

@SleepySwords I tried and still the refactoring doesn't apply

;; Move cursor to a capture ("@foo") to highlight matches in the source buffer.
;; Completion for grammar nodes is available (:help compl-omni)


(call_expression
  function: (identifier) @function
  arguments: (argument_list (_) @parameter.inner)
)

(function_declaration
  name: (identifier)
  parameters: (parameter_list (_) @parameter.inner)
)

(call_expression
  function: (selector_expression 
              field: (field_identifier) @method
              )
  arguments: (argument_list (_) @parameter.inner)
)

I will try to debug where things are going wrong.

[SleepySwords]

Hmmm, that is a bit strange, it seems to be working for me. Could you send how you setup the plugin?

It is probably worth noting that I did push a change today, which prints a message and returns if a user is not hovering over a function name when running the command. The change does use method_name and function_name rather than method and function. However, it should still work without those captures, so it seems very odd.