From bcedaebe4944b242d7ffbee27591170e22cbc0a9 Mon Sep 17 00:00:00 2001 From: inhies Date: Thu, 3 Nov 2016 11:50:03 -0600 Subject: Add Go Report Card badge --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 9096f8b..078474d 100644 --- a/README.md +++ b/README.md @@ -2,7 +2,7 @@ Package hue interfaces Philips Hue devices to control lights, scenes, schedules, and groups. [![GoDoc](https://camo.githubusercontent.com/b3b2a2b7fad4e76052830945cd839a3bba5be723/687474703a2f2f696d672e736869656c64732e696f2f62616467652f676f646f632d7265666572656e63652d3532373242342e706e67)](https://godoc.org/github.com/Collinux/GoHue) - +[![Go Report Card](https://goreportcard.com/badge/github.com/Collinux/GoHue)](https://goreportcard.com/report/github.com/Collinux/GoHue) ## Installation ``` go get github.com/collinux/gohue -- cgit v1.2.3 From abcaf8ba7f9caf56897360a5a2d7754c7b27f82e Mon Sep 17 00:00:00 2001 From: inhies Date: Thu, 3 Nov 2016 11:53:03 -0600 Subject: Go fmt bridge.go --- bridge.go | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/bridge.go b/bridge.go index 2a6fb26..9502af7 100644 --- a/bridge.go +++ b/bridge.go @@ -92,17 +92,17 @@ func (bridge *Bridge) Put(path string, params interface{}) ([]byte, io.Reader, e // a body formatted with parameters (in a generic interface). // If `params` is nil then it will send an empty body with the post request. func (bridge *Bridge) Post(path string, params interface{}) ([]byte, io.Reader, error) { - // Add the params to the request or allow an empty body - request := []byte{} - if params != nil { - reqBody, err := json.Marshal(params) - if err != nil { - err = errors.New("Error: Unable to add POST body parameters due to json marshal error.") - log.Println(err) - return []byte{}, nil, err - } - request = reqBody - } + // Add the params to the request or allow an empty body + request := []byte{} + if params != nil { + reqBody, err := json.Marshal(params) + if err != nil { + err = errors.New("Error: Unable to add POST body parameters due to json marshal error.") + log.Println(err) + return []byte{}, nil, err + } + request = reqBody + } // Send the request and handle the response uri := fmt.Sprintf("http://" + bridge.IPAddress + path) resp, err := http.Post(uri, "text/json", bytes.NewReader(request)) @@ -111,7 +111,7 @@ func (bridge *Bridge) Post(path string, params interface{}) ([]byte, io.Reader, log.Println(err) return []byte{}, nil, err } - return HandleResponse(resp) + return HandleResponse(resp) } // Bridge.Delete sends an http DELETE to the bridge @@ -134,7 +134,7 @@ func (bridge *Bridge) Delete(path string) error { // and invalid return types. func HandleResponse(resp *http.Response) ([]byte, io.Reader, error) { body, err := ioutil.ReadAll(resp.Body) - defer resp.Body.Close() + defer resp.Body.Close() if err != nil { trace("Error parsing bridge description xml.", nil) return []byte{}, nil, err @@ -162,11 +162,11 @@ func FindBridges() ([]Bridge, error) { log.Fatal(err) return []Bridge{}, err } - bridges := []Bridge{} - err = json.Unmarshal(body, &bridges) - if err != nil { - return []Bridge{}, errors.New("Unable to parse FindBridges response. ") - } + bridges := []Bridge{} + err = json.Unmarshal(body, &bridges) + if err != nil { + return []Bridge{}, errors.New("Unable to parse FindBridges response. ") + } return bridges, nil } -- cgit v1.2.3 From a712f307fe27070aa449452091553acf85e3d8f8 Mon Sep 17 00:00:00 2001 From: inhies Date: Thu, 3 Nov 2016 12:02:34 -0600 Subject: Fix complaints --- light.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/light.go b/light.go index fc1229e..fcf54fe 100644 --- a/light.go +++ b/light.go @@ -88,7 +88,6 @@ func (light *Light) Toggle() error { } else { return light.On() } - return nil } // Light.Delete removes the light from the @@ -185,7 +184,7 @@ func (light *Light) Dim(percent int) error { newBri := uint8(originalBri - uint8(decreaseBri)) if newBri < 0 { newBri = 0 - log.Println("Light.Dim state set under 0%, setting brightness to 0. ") + log.Printf("Light.Dim state set under 0%%, setting brightness to 0. ") } lightState := LightState{On: true, Bri: newBri} err := light.SetState(lightState) @@ -222,7 +221,7 @@ func (light *Light) Brighten(percent int) error { newBri := uint8(originalBri + uint8(increaseBri)) if newBri > 254 { // LightState.Bri must be between 1 and 254 inclusive newBri = 254 - log.Println("Light.Brighten state set over 100%, setting brightness to 100%. ") + log.Printf("Light.Brighten state set over 100%%, setting brightness to 100%%. ") } lightState := LightState{On: true, Bri: newBri} err := light.SetState(lightState) -- cgit v1.2.3