RESTClient not accepting query string about go-gh

[gennaro-tedesco]

Thank you very much for your exhaustive reply, it works fine!

[mislav]

You can add query parameters to the "search/repositories" string by constructing a url.Values object and serializing it to string using its Encode() method.

The argument to client.Get(path) is the path component of the URL with optional query parameters separated with ?.

[gennaro-tedesco]

Thank you for your quick reply. However, when you say:

The argument to client.Get(path) is the path component of the URL with optional query parameters separated with ?.

would this mean that at the end of the day one must pass a whole string of the form

search/repositories?name=<name>?language=lang

to client.Get()? Because this looks different from what the query string for gh API /search/repositories -f q='...' is (so it is a little hard to understand what the right grammar is).

[mislav]

When you do

gh api -X GET search/repositories -f q1=foo -f q2=bar

that's equivalent to

gh api search/repositories?q1=foo&q2=bar

with the added bonus that you don't have to worry about encoding all the special characters in the query string properly, since that's handled by gh api.

Similarly, you could do:

client.Get("path?q1=foo&q2=bar")

and that will work, but I wouldn't recommend it because, again, you'd have to worry about properly encoding query values. Instead, you should do:

searchTerm := "<name>"
repoLanguage := "go"
myQuery := url.Values{}
myQuery.Add("q", fmt.Sprintf("%s in:name language:%s", searchTerm, repoLanguage))
myQuery.Add("per_page", "30")
client.Get("path?"+myQuery.Encode())