re-order rcfile parsing

lgtmco · Apr 19, 2016 · 66ef421 · 66ef421
1 parent 9cc16ed
commit 66ef421
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/web/hook.go b/web/hook.go
@@ -37,6 +37,14 @@ func Hook(c *gin.Context) {
 		return
 	}
 
+	rcfile, _ := remote.GetContents(c, user, repo, ".lgtm")
+	config, err := model.ParseConfig(rcfile)
+	if err != nil {
+		log.Errorf("Error parsing .lgtm file for %s. %s", repo.Slug, err)
+		c.String(500, "Error parsing .lgtm file. %s.", err)
+		return
+	}
+
 	// THIS IS COMPLETELY DUPLICATED IN THE API SECTION. NOT IDEAL
 	file, err := remote.GetContents(c, user, repo, "MAINTAINERS")
 	if err != nil {
@@ -62,14 +70,6 @@ func Hook(c *gin.Context) {
 		return
 	}
 
-	rcfile, _ := remote.GetContents(c, user, repo, ".lgtm")
-	config, err := model.ParseConfig(rcfile)
-	if err != nil {
-		log.Errorf("Error parsing .lgtm file for %s. %s", repo.Slug, err)
-		c.String(500, "Error parsing .lgtm file. %s.", err)
-		return
-	}
-
 	comments, err := remote.GetComments(c, user, repo, hook.Issue.Number)
 	if err != nil {
 		log.Errorf("Error retrieving comments for %s pr %d. %s", repo.Slug, hook.Issue.Number, err)