Merge pull request #9 from mom0tomo/fix_error_handling

Fix missing err processing and missing body closing

Author: moutonjeremy

Diff for: metrics/get_billable_from_github.go

@@ -51,10 +51,10 @@ func GetBillableFromGithub() {
 				req, _ := http.NewRequest("GET", "https://"+config.Github.ApiUrl+"/repos/"+repo+"/actions/workflows/"+strconv.Itoa(k)+"/timing", nil)
 				req.Header.Set("Authorization", "token "+config.Github.Token)
 				resp, err := client.Do(req)
-				defer resp.Body.Close()
 				if err != nil {
 					log.Fatal(err)
 				}
+				defer resp.Body.Close()
 				if resp.StatusCode != 200 {
 					log.Fatalf("the status code returned by the server for workflows in repo %s is different from 200: %d", repo, resp.StatusCode)
 				}

Diff for: metrics/get_runners_from_github.go

@@ -47,6 +47,7 @@ func GetRunnersFromGithub() {
 			if err != nil {
 				log.Fatal(err)
 			}
+			defer resp.Body.Close()
 			if resp.StatusCode != 200 {
 				log.Fatalf("the status code returned by the server for runners in repo %s is different from 200: %d", repo, resp.StatusCode)
 			}

Diff for: metrics/worflows_cache.go

@@ -36,6 +36,7 @@ func WorkflowsCache() {
 			if err != nil {
 				log.Fatal(err)
 			}
+
 			if resp.StatusCode != 200 {
 				log.Fatalf("the status code returned by the server for workflows in repo %s is different from 200: %d", repo, resp.StatusCode)
 			}