Fix CI

Author: doriable

check/check_service_handler.go

@@ -257,7 +257,7 @@ func (c *checkServiceHandler) getRulesAndNextPageToken(pageSize int, pageToken s
 		pageSize = defaultPageSize
 	}
 	resultRules := make([]Rule, 0, len(c.rules)-index)
-	for i := 0; i < pageSize; i++ {
+	for range pageSize {
 		if index >= len(c.rules) {
 			break
 		}
@@ -284,7 +284,7 @@ func (c *checkServiceHandler) getCategoriesAndNextPageToken(pageSize int, pageTo
 		pageSize = defaultPageSize
 	}
 	resultCategories := make([]Category, 0, len(c.categories)-index)
-	for i := 0; i < pageSize; i++ {
+	for range pageSize {
 		if index >= len(c.categories) {
 			break
 		}

check/checktest/checktest.go

@@ -380,7 +380,7 @@ func unusedDependencyIndexesForFilePathToUnusedDependencyFilePaths(
 		return unusedDependencyIndexes
 	}
 	dependencyFilePaths := fileDescriptorProto.GetDependency()
-	for i := 0; i < len(dependencyFilePaths); i++ {
+	for i := range len(dependencyFilePaths) {
 		if _, ok := unusedDependencyFilePaths[dependencyFilePaths[i]]; ok {
 			unusedDependencyIndexes = append(unusedDependencyIndexes, int32(i))
 		}

check/checkutil/util.go

@@ -186,7 +186,7 @@ func forEachFileImport(
 	f func(protoreflect.FileImport) error,
 ) error {
 	fileImports := fileDescriptor.Imports()
-	for i := 0; i < fileImports.Len(); i++ {
+	for i := range fileImports.Len() {
 		if err := f(fileImports.Get(i)); err != nil {
 			return err
 		}
@@ -199,13 +199,13 @@ func forEachEnum(
 	f func(protoreflect.EnumDescriptor) error,
 ) error {
 	enums := container.Enums()
-	for i := 0; i < enums.Len(); i++ {
+	for i := range enums.Len() {
 		if err := f(enums.Get(i)); err != nil {
 			return err
 		}
 	}
 	messages := container.Messages()
-	for i := 0; i < messages.Len(); i++ {
+	for i := range messages.Len() {
 		// Nested enums.
 		if err := forEachEnum(messages.Get(i), f); err != nil {
 			return err
@@ -219,7 +219,7 @@ func forEachEnumValue(
 	f func(protoreflect.EnumValueDescriptor) error,
 ) error {
 	enumValues := enumDescriptor.Values()
-	for i := 0; i < enumValues.Len(); i++ {
+	for i := range enumValues.Len() {
 		if err := f(enumValues.Get(i)); err != nil {
 			return err
 		}
@@ -232,7 +232,7 @@ func forEachMessage(
 	f func(protoreflect.MessageDescriptor) error,
 ) error {
 	messages := container.Messages()
-	for i := 0; i < messages.Len(); i++ {
+	for i := range messages.Len() {
 		messageDescriptor := messages.Get(i)
 		if err := f(messageDescriptor); err != nil {
 			return err
@@ -253,13 +253,13 @@ func forEachField(
 		container,
 		func(messageDescriptor protoreflect.MessageDescriptor) error {
 			fields := messageDescriptor.Fields()
-			for i := 0; i < fields.Len(); i++ {
+			for i := range fields.Len() {
 				if err := f(fields.Get(i)); err != nil {
 					return err
 				}
 			}
 			extensions := messageDescriptor.Extensions()
-			for i := 0; i < extensions.Len(); i++ {
+			for i := range extensions.Len() {
 				if err := f(extensions.Get(i)); err != nil {
 					return err
 				}
@@ -270,7 +270,7 @@ func forEachField(
 		return err
 	}
 	extensions := container.Extensions()
-	for i := 0; i < extensions.Len(); i++ {
+	for i := range extensions.Len() {
 		if err := f(extensions.Get(i)); err != nil {
 			return err
 		}
@@ -283,7 +283,7 @@ func forEachOneof(
 	f func(protoreflect.OneofDescriptor) error,
 ) error {
 	oneofs := messageDescriptor.Oneofs()
-	for i := 0; i < oneofs.Len(); i++ {
+	for i := range oneofs.Len() {
 		if err := f(oneofs.Get(i)); err != nil {
 			return err
 		}
@@ -296,7 +296,7 @@ func forEachService(
 	f func(protoreflect.ServiceDescriptor) error,
 ) error {
 	services := fileDescriptor.Services()
-	for i := 0; i < services.Len(); i++ {
+	for i := range services.Len() {
 		if err := f(services.Get(i)); err != nil {
 			return err
 		}
@@ -309,7 +309,7 @@ func forEachMethod(
 	f func(protoreflect.MethodDescriptor) error,
 ) error {
 	methods := serviceDescriptor.Methods()
-	for i := 0; i < methods.Len(); i++ {
+	for i := range methods.Len() {
 		if err := f(methods.Get(i)); err != nil {
 			return err
 		}

check/client_test.go

@@ -134,7 +134,7 @@ func TestClientListRulesCount(t *testing.T) {
 func testClientListRulesCount(t *testing.T, count int) {
 	require.True(t, count < 10000, "count must be less than 10000 for sorting to work properly in this test")
 	ruleSpecs := make([]*RuleSpec, count)
-	for i := 0; i < count; i++ {
+	for i := range count {
 		ruleSpecs[i] = &RuleSpec{
 			ID:      fmt.Sprintf("RULE%05d", i),
 			Purpose: fmt.Sprintf("Test RULE%05d.", i),
@@ -150,7 +150,7 @@ func testClientListRulesCount(t *testing.T, count int) {
 	rules, err := client.ListRules(context.Background())
 	require.NoError(t, err)
 	require.Equal(t, count, len(rules))
-	for i := 0; i < count; i++ {
+	for i := range count {
 		require.Equal(t, ruleSpecs[i].ID, rules[i].ID())
 	}
 }

check/internal/example/gen/acme/option/v1/option.pb.go

@@ -31,7 +31,7 @@ func TestParallelizeSimple(t *testing.T) {
 	numJobs := 10
 	var executed atomic.Int64
 	jobs := make([]func(context.Context) error, 0, numJobs)
-	for i := 0; i < numJobs; i++ {
+	for range numJobs {
 		jobs = append(
 			jobs,
 			func(context.Context) error {
@@ -51,7 +51,7 @@ func TestParallelizeImmediateCancellation(t *testing.T) {
 	numJobs := 10
 	var executed atomic.Int64
 	jobs := make([]func(context.Context) error, 0, numJobs)
-	for i := 0; i < numJobs; i++ {
+	for range numJobs {
 		jobs = append(
 			jobs,
 			func(context.Context) error {

internal/pkg/thread/thread_test.go

@@ -301,7 +301,7 @@ func valueToProtoValue(value any) (*optionv1.Value, error) {
 			}, nil
 		}
 		values := make([]*optionv1.Value, reflectValue.Len())
-		for i := 0; i < reflectValue.Len(); i++ {
+		for i := range reflectValue.Len() {
 			subValue, err := valueToProtoValue(reflectValue.Index(i).Interface())
 			if err != nil {
 				return nil, err