move isURL

vilmibm · vilmibm · commit 70c948a7519e · 2020-07-23T11:23:45.000-05:00
diff --git a/command/repo.go b/command/repo.go
@@ -353,10 +353,6 @@ func repoCreate(cmd *cobra.Command, args []string) error {
 	return nil
 }
 
-func isURL(arg string) bool {
-	return strings.HasPrefix(arg, "http:/") || strings.HasPrefix(arg, "https:/")
-}
-
 var Since = func(t time.Time) time.Duration {
 	return time.Since(t)
 }
@@ -390,7 +386,7 @@ func repoFork(cmd *cobra.Command, args []string) error {
 	} else {
 		repoArg := args[0]
 
-		if isURL(repoArg) {
+		if utils.IsURL(repoArg) {
 			parsedURL, err := url.Parse(repoArg)
 			if err != nil {
 				return fmt.Errorf("did not understand argument: %w", err)
diff --git a/pkg/cmd/repo/view/view.go b/pkg/cmd/repo/view/view.go
@@ -71,7 +71,7 @@ func viewRun(opts *ViewOptions) error {
 			return err
 		}
 	} else {
-		if isURL(opts.RepoArg) {
+		if utils.IsURL(opts.RepoArg) {
 			parsedURL, err := url.Parse(opts.RepoArg)
 			if err != nil {
 				return fmt.Errorf("did not understand argument: %w", err)
@@ -188,10 +188,6 @@ func isMarkdownFile(filename string) bool {
 }
 
 // TODO COPYPASTA FROM command; CONSIDER FOR cmdutil?
-func isURL(arg string) bool {
-	return strings.HasPrefix(arg, "http:/") || strings.HasPrefix(arg, "https:/")
-}
-
 func displayURL(urlStr string) string {
 	u, err := url.Parse(urlStr)
 	if err != nil {
diff --git a/utils/utils.go b/utils/utils.go
@@ -102,3 +102,7 @@ var StopSpinner = func(s *spinner.Spinner) {
 func Spinner(w io.Writer) *spinner.Spinner {
 	return spinner.New(spinner.CharSets[11], 400*time.Millisecond, spinner.WithWriter(w))
 }
+
+func IsURL(s string) bool {
+	return strings.HasPrefix(s, "http:/") || strings.HasPrefix(s, "https:/")
+}

Original file line number	Diff line number	Diff line change
`@@ -102,3 +102,7 @@ var StopSpinner = func(s *spinner.Spinner) {`
`102`	`102`	`func Spinner(w io.Writer) *spinner.Spinner {`
`103`	`103`	`return spinner.New(spinner.CharSets[11], 400*time.Millisecond, spinner.WithWriter(w))`
`104`	`104`	`}`
	`105`	`+`
	`106`	`+func IsURL(s string) bool {`
	`107`	`+ return strings.HasPrefix(s, "http:/") \|\| strings.HasPrefix(s, "https:/")`
	`108`	`+}`