Fix utility function scopes

doi-t · doi-t · commit 46a632c8f643 · 2020-03-18T04:52:03.000+09:00
diff --git a/command/issue.go b/command/issue.go
@@ -248,7 +248,7 @@ func issueView(cmd *cobra.Command, args []string) error {
 
 }
 
-func IssueStateTitleWithColor(state string) string {
+func issueStateTitleWithColor(state string) string {
 	colorFunc := utils.ColorFuncForState(state)
 	return colorFunc(strings.Title(strings.ToLower(state)))
 }
@@ -258,7 +258,7 @@ func printIssuePreview(out io.Writer, issue *api.Issue) error {
 	ago := now.Sub(issue.CreatedAt)
 
 	fmt.Fprintln(out, utils.Bold(issue.Title))
-	fmt.Fprintf(out, "%s", IssueStateTitleWithColor(issue.State))
+	fmt.Fprintf(out, "%s", issueStateTitleWithColor(issue.State))
 	fmt.Fprintln(out, utils.Gray(fmt.Sprintf(
 		" • %s opened %s • %s",
 		issue.Author.Login,
diff --git a/command/pr.go b/command/pr.go
@@ -228,7 +228,7 @@ func prList(cmd *cobra.Command, args []string) error {
 	return nil
 }
 
-func PRStateTitleWithColor(pr api.PullRequest) string {
+func prStateTitleWithColor(pr api.PullRequest) string {
 	prStateColorFunc := colorFuncForPR(pr)
 	return prStateColorFunc(strings.Title(strings.ToLower(pr.State)))
 }
@@ -302,7 +302,7 @@ func prView(cmd *cobra.Command, args []string) error {
 
 func printPrPreview(out io.Writer, pr *api.PullRequest) error {
 	fmt.Fprintln(out, utils.Bold(pr.Title))
-	fmt.Fprintf(out, "%s", PRStateTitleWithColor(*pr))
+	fmt.Fprintf(out, "%s", prStateTitleWithColor(*pr))
 	fmt.Fprintln(out, utils.Gray(fmt.Sprintf(
 		" • %s wants to merge %s into %s from %s",
 		pr.Author.Login,
@@ -427,7 +427,7 @@ func printPrs(w io.Writer, totalCount int, prs ...api.PullRequest) {
 				fmt.Fprintf(w, " - %s", utils.Green("Approved"))
 			}
 		} else {
-			fmt.Fprintf(w, " - %s", PRStateTitleWithColor(pr))
+			fmt.Fprintf(w, " - %s", prStateTitleWithColor(pr))
 		}
 
 		fmt.Fprint(w, "\n")

Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ func issueView(cmd *cobra.Command, args []string) error {`
`248`	`248`
`249`	`249`	`}`
`250`	`250`
`251`		`-func IssueStateTitleWithColor(state string) string {`
	`251`	`+func issueStateTitleWithColor(state string) string {`
`252`	`252`	`colorFunc := utils.ColorFuncForState(state)`
`253`	`253`	`return colorFunc(strings.Title(strings.ToLower(state)))`
`254`	`254`	`}`
`@@ -258,7 +258,7 @@ func printIssuePreview(out io.Writer, issue *api.Issue) error {`
`258`	`258`	`ago := now.Sub(issue.CreatedAt)`
`259`	`259`
`260`	`260`	`fmt.Fprintln(out, utils.Bold(issue.Title))`
`261`		`- fmt.Fprintf(out, "%s", IssueStateTitleWithColor(issue.State))`
	`261`	`+ fmt.Fprintf(out, "%s", issueStateTitleWithColor(issue.State))`
`262`	`262`	`fmt.Fprintln(out, utils.Gray(fmt.Sprintf(`
`263`	`263`	`" • %s opened %s • %s",`
`264`	`264`	`issue.Author.Login,`
Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,7 @@ func prList(cmd *cobra.Command, args []string) error {`
`228`	`228`	`return nil`
`229`	`229`	`}`
`230`	`230`
`231`		`-func PRStateTitleWithColor(pr api.PullRequest) string {`
	`231`	`+func prStateTitleWithColor(pr api.PullRequest) string {`
`232`	`232`	`prStateColorFunc := colorFuncForPR(pr)`
`233`	`233`	`return prStateColorFunc(strings.Title(strings.ToLower(pr.State)))`
`234`	`234`	`}`
`@@ -302,7 +302,7 @@ func prView(cmd *cobra.Command, args []string) error {`
`302`	`302`
`303`	`303`	`func printPrPreview(out io.Writer, pr *api.PullRequest) error {`
`304`	`304`	`fmt.Fprintln(out, utils.Bold(pr.Title))`
`305`		`- fmt.Fprintf(out, "%s", PRStateTitleWithColor(*pr))`
	`305`	`+ fmt.Fprintf(out, "%s", prStateTitleWithColor(*pr))`
`306`	`306`	`fmt.Fprintln(out, utils.Gray(fmt.Sprintf(`
`307`	`307`	`" • %s wants to merge %s into %s from %s",`
`308`	`308`	`pr.Author.Login,`
`@@ -427,7 +427,7 @@ func printPrs(w io.Writer, totalCount int, prs ...api.PullRequest) {`
`427`	`427`	`fmt.Fprintf(w, " - %s", utils.Green("Approved"))`
`428`	`428`	`}`
`429`	`429`	`} else {`
`430`		`- fmt.Fprintf(w, " - %s", PRStateTitleWithColor(pr))`
	`430`	`+ fmt.Fprintf(w, " - %s", prStateTitleWithColor(pr))`
`431`	`431`	`}`
`432`	`432`
`433`	`433`	`fmt.Fprint(w, "\n")`