diff options
author | Michael Muré <michael.mure@consensys.net> | 2018-12-17 13:05:05 +0100 |
---|---|---|
committer | Michael Muré <michael.mure@consensys.net> | 2018-12-17 13:05:05 +0100 |
commit | 47b2aa4cc1c20ae37d846da4ea71e09c0c918192 (patch) | |
tree | 143f7d7f5e823afeb0b9616b39996fc37e63c336 | |
parent | d26b1d08c71247dbc66a44e11a51f6b382ca9868 (diff) | |
download | git-bug-47b2aa4cc1c20ae37d846da4ea71e09c0c918192.tar.gz |
launchpad: minor cleaning
-rw-r--r-- | bridge/launchpad/config.go | 12 | ||||
-rw-r--r-- | bridge/launchpad/launchpad_api.go | 7 |
2 files changed, 12 insertions, 7 deletions
diff --git a/bridge/launchpad/config.go b/bridge/launchpad/config.go index 8469dbd3..11a465be 100644 --- a/bridge/launchpad/config.go +++ b/bridge/launchpad/config.go @@ -25,6 +25,14 @@ func (*Launchpad) Configure(repo repository.RepoCommon) (core.Configuration, err return conf, nil } +func (*Launchpad) ValidateConfig(conf core.Configuration) error { + if _, ok := conf[keyProject]; !ok { + return fmt.Errorf("missing %s key", keyProject) + } + + return nil +} + func promptProjectName() (string, error) { for { fmt.Print("Launchpad project name: ") @@ -44,7 +52,3 @@ func promptProjectName() (string, error) { return line, nil } } - -func (*Launchpad) ValidateConfig(conf core.Configuration) error { - return nil -} diff --git a/bridge/launchpad/launchpad_api.go b/bridge/launchpad/launchpad_api.go index 09e02bc5..849ef925 100644 --- a/bridge/launchpad/launchpad_api.go +++ b/bridge/launchpad/launchpad_api.go @@ -129,11 +129,12 @@ func (lapi *launchpadAPI) SearchTasks(project string) ([]LPBug, error) { return nil, err } - defer resp.Body.Close() - var result launchpadAnswer - if err := json.NewDecoder(resp.Body).Decode(&result); err != nil { + err = json.NewDecoder(resp.Body).Decode(&result) + _ = resp.Body.Close() + + if err != nil { return nil, err } |