Merge branch 'main' into k/partial-revert

Author: keegancsmith

cmd/src/extsvc_list.go

@@ -29,7 +29,7 @@ Examples:
 		fmt.Println(usage)
 	}
 	var (
-		firstFlag  = flagSet.Int("first", -1, "Return only the first n external services. (use -1 for unlimited)")
+		firstFlag  = flagSet.Int("first", 1000, "Return only the first n external services.")
 		formatFlag = flagSet.String("f", "", `Format for the output, using the syntax of Go package text/template. (e.g. "{{.|json}}")`)
 		apiFlags   = api.NewFlags(flagSet)
 	)
@@ -40,9 +40,6 @@ Examples:
 		}
 
 		first := *firstFlag
-		if first == -1 {
-			first = 9999999 // GraphQL API doesn't support negative for unlimited query
-		}
 
 		var formatStr string
 		if *formatFlag != "" {

cmd/src/orgs_list.go

@@ -16,10 +16,6 @@ Examples:
 
     	$ src orgs list
 
-  List *all* organizations (may be slow!):
-
-    	$ src orgs list -first='-1'
-
   List organizations whose names match the query:
 
     	$ src orgs list -query='myquery'
@@ -33,7 +29,7 @@ Examples:
 		fmt.Println(usage)
 	}
 	var (
-		firstFlag  = flagSet.Int("first", 1000, "Returns the first n organizations from the list. (use -1 for unlimited)")
+		firstFlag  = flagSet.Int("first", 1000, "Returns the first n organizations from the list.")
 		queryFlag  = flagSet.String("query", "", `Returns organizations whose names match the query. (e.g. "alice")`)
 		formatFlag = flagSet.String("f", "{{.Name}}", `Format for the output, using the syntax of Go package text/template. (e.g. "{{.ID}}: {{.Name}} ({{.DisplayName}})" or "{{.|json}}")`)
 		apiFlags   = api.NewFlags(flagSet)

cmd/src/repos_list.go

@@ -165,10 +165,6 @@ Examples:
 
     	$ src repos list -f '{{.|json}}'
 
-  List *all* repositories (may be slow!):
-
-    	$ src repos list -first='-1'
-
   List repositories whose names match the query:
 
     	$ src repos list -query='myquery'
@@ -182,7 +178,7 @@ Examples:
 		fmt.Println(usage)
 	}
 	var (
-		firstFlag = flagSet.Int("first", 1000, "Returns the first n repositories from the list. (use -1 for unlimited)")
+		firstFlag = flagSet.Int("first", 1000, "Returns the first n repositories from the list.")
 		queryFlag = flagSet.String("query", "", `Returns repositories whose names match the query. (e.g. "myorg/")`)
 		// TODO: add support for "names" field.
 		clonedFlag           = flagSet.Bool("cloned", true, "Include cloned repositories.")

cmd/src/users_list.go

@@ -16,10 +16,6 @@ Examples:
 
     	$ src users list
 
-  List *all* users (may be slow!):
-
-    	$ src users list -first='-1'
-
   List users whose names match the query:
 
     	$ src users list -query='myquery'
@@ -37,7 +33,7 @@ Examples:
 		fmt.Println(usage)
 	}
 	var (
-		firstFlag  = flagSet.Int("first", 1000, "Returns the first n users from the list. (use -1 for unlimited)")
+		firstFlag  = flagSet.Int("first", 1000, "Returns the first n users from the list.")
 		queryFlag  = flagSet.String("query", "", `Returns users whose names match the query. (e.g. "alice")`)
 		tagFlag    = flagSet.String("tag", "", `Returns users with the given tag.`)
 		formatFlag = flagSet.String("f", "{{.Username}}", `Format for the output, using the syntax of Go package text/template. (e.g. "{{.ID}}: {{.Username}} ({{.DisplayName}})" or "{{.|json}}")`)