Remove rawQuery argument to newRequest() as per linter

Author: EmmEff

client/build.go

@@ -23,7 +23,7 @@ func (c *Client) Submit(ctx context.Context, br BuildRequest) (bi BuildInfo, err
 		return
 	}
 
-	req, err := c.newRequest(http.MethodPost, "/v1/build", "", bytes.NewReader(b))
+	req, err := c.newRequest(http.MethodPost, "/v1/build", bytes.NewReader(b))
 	if err != nil {
 		return
 	}
@@ -47,7 +47,7 @@ func (c *Client) Submit(ctx context.Context, br BuildRequest) (bi BuildInfo, err
 // Cancel cancels an existing build. The context controls the lifetime of the
 // request.
 func (c *Client) Cancel(ctx context.Context, buildID string) error {
-	req, err := c.newRequest(http.MethodPut, fmt.Sprintf("/v1/build/%s/_cancel", buildID), "", nil)
+	req, err := c.newRequest(http.MethodPut, fmt.Sprintf("/v1/build/%s/_cancel", buildID), nil)
 	if err != nil {
 		return err
 	}

client/client.go

@@ -88,10 +88,9 @@ func New(cfg *Config) (c *Client, err error) {
 }
 
 // newRequest returns a new Request given a method, path, query, and optional body.
-func (c *Client) newRequest(method, path, rawQuery string, body io.Reader) (r *http.Request, err error) {
+func (c *Client) newRequest(method, path string, body io.Reader) (r *http.Request, err error) {
 	u := c.BaseURL.ResolveReference(&url.URL{
-		Path:     path,
-		RawQuery: rawQuery,
+		Path: path,
 	})
 
 	r, err = http.NewRequest(method, u.String(), body)

client/status.go

@@ -14,7 +14,7 @@ import (
 
 // GetStatus gets the status of a build from the Build Service by build ID
 func (c *Client) GetStatus(ctx context.Context, buildID string) (bi BuildInfo, err error) {
-	req, err := c.newRequest(http.MethodGet, "/v1/build/"+buildID, "", nil)
+	req, err := c.newRequest(http.MethodGet, "/v1/build/"+buildID, nil)
 	if err != nil {
 		return
 	}

client/version.go

@@ -22,7 +22,7 @@ type VersionInfo struct {
 // GetVersion gets version information from the build service. The context
 // controls the lifetime of the request.
 func (c *Client) GetVersion(ctx context.Context) (vi VersionInfo, err error) {
-	req, err := c.newRequest(http.MethodGet, pathVersion, "", nil)
+	req, err := c.newRequest(http.MethodGet, pathVersion, nil)
 	if err != nil {
 		return VersionInfo{}, err
 	}