Clean up some colourizing switch statements

wtfutil · Jun 16, 2018 · d74c4eb · d74c4eb
1 parent 66b6947
commit d74c4eb
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 15 deletions.
diff --git a/jenkins/widget.go b/jenkins/widget.go
@@ -67,16 +67,12 @@ func (widget *Widget) contentFrom(view *View) string {
 }
 
 func (widget *Widget) jobColor(job *Job) string {
-	var color string
-
 	switch job.Color {
 	case "blue":
-		color = "green"
+		return "blue"
 	case "red":
-		color = "red"
+		return "red"
 	default:
-		color = "white"
+		return "white"
 	}
-
-	return color
 }
diff --git a/jira/widget.go b/jira/widget.go
@@ -66,20 +66,16 @@ func (widget *Widget) contentFrom(searchResult *SearchResult) string {
 }
 
 func (widget *Widget) issueTypeColor(issue *Issue) string {
-	var color string
-
 	switch issue.IssueFields.IssueType.Name {
 	case "Bug":
-		color = "red"
+		return "red"
 	case "Story":
-		color = "blue"
+		return "blue"
 	case "Task":
-		color = "orange"
+		return "orange"
 	default:
-		color = "white"
+		return "white"
 	}
-
-	return color
 }
 
 func getProjects() []string {