(chore)get rid of logrus from all core structs except CLI entries. fix schedule override command

addon.go

@@ -2,11 +2,8 @@ package pagerduty
 
 import (
 	"fmt"
-	"io/ioutil"
-	"net/http"
-
-	log "github.com/Sirupsen/logrus"
 	"github.com/google/go-querystring/query"
+	"net/http"
 )
 
 // Addon is a third-party add-on to PagerDuty's UI.
@@ -55,10 +52,6 @@ func (c *Client) InstallAddon(a Addon) (*Addon, error) {
 		return nil, err
 	}
 	if resp.StatusCode != http.StatusCreated {
-		ct, rErr := ioutil.ReadAll(resp.Body)
-		if rErr == nil {
-			log.Debug(string(ct))
-		}
 		return nil, fmt.Errorf("Failed to create. HTTP Status code: %d", resp.StatusCode)
 	}
 	return getAddonFromResponse(c, resp)

client.go

@@ -6,8 +6,6 @@ import (
 	"fmt"
 	"io"
 	"net/http"
-
-	log "github.com/Sirupsen/logrus"
 )
 
 const (
@@ -73,7 +71,6 @@ func (c *Client) post(path string, payload interface{}) (*http.Response, error)
 	if err != nil {
 		return nil, err
 	}
-	log.Debugln(string(data))
 	return c.do("POST", path, bytes.NewBuffer(data))
 }
 
@@ -83,7 +80,6 @@ func (c *Client) get(path string) (*http.Response, error) {
 
 func (c *Client) do(method, path string, body io.Reader) (*http.Response, error) {
 	endpoint := apiEndpoint + path
-	log.Debugln("Endpoint:", endpoint)
 	req, _ := http.NewRequest(method, endpoint, body)
 	req.Header.Set("Accept", "application/vnd.pagerduty+json;version=2")
 	req.Header.Set("Content-Type", "application/json")

command/schedule_override_create.go

@@ -41,7 +41,7 @@ func (c *ScheduleOverrideCreate) Run(args []string) int {
 		return -1
 	}
 	client := c.Meta.Client()
-	var o pagerduty.Overrides
+	var o pagerduty.Override
 	if len(flags.Args()) != 2 {
 		log.Error("Please specify input json file")
 		return -1

escalation_policy.go

@@ -2,9 +2,8 @@ package pagerduty
 
 import (
 	"fmt"
-	"net/http"
-
 	"github.com/google/go-querystring/query"
+	"net/http"
 )
 
 const (

event.go

@@ -5,8 +5,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-
-	log "github.com/Sirupsen/logrus"
 )
 
 const eventEndPoint = "https://events.pagerduty.com/generic/2010-04-15/create_event.json"
@@ -32,12 +30,10 @@ type EventResponse struct {
 
 // CreateEvent sends PagerDuty an event to report, acknowledge, or resolve a problem.
 func CreateEvent(e Event) (*EventResponse, error) {
-	log.Debugln("Endpoint:", eventEndPoint)
 	data, err := json.Marshal(e)
 	if err != nil {
 		return nil, err
 	}
-	log.Debugln(string(data))
 	req, _ := http.NewRequest("POST", eventEndPoint, bytes.NewBuffer(data))
 	req.Header.Set("Content-Type", "application/json")
 	resp, err := http.DefaultClient.Do(req)

incident.go

@@ -2,7 +2,6 @@ package pagerduty
 
 import (
 	"fmt"
-
 	"github.com/google/go-querystring/query"
 )
 

log_entry.go

@@ -2,7 +2,6 @@ package pagerduty
 
 import (
 	"fmt"
-
 	"github.com/google/go-querystring/query"
 )
 

maintenance_window.go

@@ -2,9 +2,8 @@ package pagerduty
 
 import (
 	"fmt"
-	"net/http"
-
 	"github.com/google/go-querystring/query"
+	"net/http"
 )
 
 // MaintenanceWindow is used to temporarily disable one or more services for a set period of time.

service.go

@@ -2,9 +2,8 @@ package pagerduty
 
 import (
 	"fmt"
-	"net/http"
-
 	"github.com/google/go-querystring/query"
+	"net/http"
 )
 
 // Integration is an endpoint (like Nagios, email, or an API call) that generates events, which are normalized and de-duplicated by PagerDuty to create incidents.

team.go

@@ -2,9 +2,8 @@ package pagerduty
 
 import (
 	"fmt"
-	"net/http"
-
 	"github.com/google/go-querystring/query"
+	"net/http"
 )
 
 // Team is a collection of users and escalation policies that represent a group of people within an organization.

user.go

@@ -2,9 +2,8 @@ package pagerduty
 
 import (
 	"fmt"
-	"net/http"
-
 	"github.com/google/go-querystring/query"
+	"net/http"
 )
 
 // ContactMethod is a way of contacting the user.