update lll tab size 4 (#4168)

Author: vhvb1989

cli/azd/.golangci.yaml

@@ -11,4 +11,5 @@ linters-settings:
     comparison: true
   lll:
     # GitHub recommended for code review
-    line-length: 125
+    line-length: 125
+    tab-width: 4

cli/azd/cmd/container.go

@@ -261,7 +261,8 @@ func registerCommonDependencies(container *ioc.NestedContainer) {
 
 	// Environment manager depends on azd context
 	container.MustRegisterSingleton(
-		func(serviceLocator ioc.ServiceLocator, azdContext *lazy.Lazy[*azdcontext.AzdContext]) *lazy.Lazy[environment.Manager] {
+		func(serviceLocator ioc.ServiceLocator,
+			azdContext *lazy.Lazy[*azdcontext.AzdContext]) *lazy.Lazy[environment.Manager] {
 			return lazy.NewLazy(func() (environment.Manager, error) {
 				azdCtx, err := azdContext.GetValue()
 				if err != nil {
@@ -415,7 +416,8 @@ func registerCommonDependencies(container *ioc.NestedContainer) {
 								return nil, &internal.ErrorWithSuggestion{
 									Err: err,
 									Suggestion: fmt.Sprintf(
-										"Set the cloud configuration by editing the 'cloud' node in the config.json file for the %s environment\n%s",
+										"Set the cloud configuration by editing the 'cloud' node in the config.json "+
+											"file for the %s environment\n%s",
 										defaultEnvName,
 										validClouds,
 									),
@@ -453,8 +455,9 @@ func registerCommonDependencies(container *ioc.NestedContainer) {
 						return cloud, nil
 					} else {
 						return nil, &internal.ErrorWithSuggestion{
-							Err:        err,
-							Suggestion: fmt.Sprintf("Set the cloud configuration using 'azd config set cloud.name <name>'.\n%s", validClouds),
+							Err: err,
+							Suggestion: fmt.Sprintf(
+								"Set the cloud configuration using 'azd config set cloud.name <name>'.\n%s", validClouds),
 						}
 					}
 				}

cli/azd/internal/cmd/provision.go

@@ -34,8 +34,9 @@ type ProvisionFlags struct {
 }
 
 const (
-	AINotValid                      = "is not valid according to the validation procedure"
-	openAIsubscriptionNoQuotaId     = "The subscription does not have QuotaId/Feature required by SKU 'S0' from kind 'OpenAI'"
+	AINotValid                  = "is not valid according to the validation procedure"
+	openAIsubscriptionNoQuotaId = "The subscription does not have QuotaId/Feature required by SKU 'S0' " +
+		"from kind 'OpenAI'"
 	responsibleAITerms              = "until you agree to Responsible AI terms for this resource"
 	specialFeatureOrQuotaIdRequired = "SpecialFeatureOrQuotaIdRequired"
 )

cli/azd/internal/telemetry/storage_test.go

@@ -347,7 +347,8 @@ func TestStorageQueue_Cleanup(t *testing.T) {
 						assert.Fail(
 							t,
 							fmt.Sprintf(
-								"Unknown remaining file found. Filename: %s, content: %s. Expected filenames: %v, expected content: %v. ",
+								"Unknown remaining file found. Filename: %s, content: %s. Expected filenames: %v, "+
+									"expected content: %v. ",
 								remainingFile.Name(),
 								string(content),
 								tt.expectedFilesRemaining,

cli/azd/internal/telemetry/uploader.go

@@ -193,7 +193,8 @@ func (u *TelemetryUploader) transmit(ctx context.Context, item *StoredItem) {
 		u.enqueueRetry(ctx, item.fileName, payload, defaultTransmitRetryDelay, attempts)
 	} else if result.CanRetry() {
 		if attempts > maxRetryCount {
-			log.Printf("failed to send %v after %d attempts, statusCode: %d\n", item.fileName, maxRetryCount, result.StatusCode)
+			log.Printf(
+				"failed to send %v after %d attempts, statusCode: %d\n", item.fileName, maxRetryCount, result.StatusCode)
 			return
 		}
 

cli/azd/internal/vsrpc/environment_service_create.go

@@ -47,8 +47,8 @@ func (s *environmentService) CreateEnvironmentAsync(
 
 	// We had thought at one point that we would introduce `ASPIRE_ENVIRONMENT` as a sibling to `ASPNETCORE_ENVIRONMENT` and
 	// `DOTNET_ENVIRONMENT` and was aspire specific. We no longer intend to do this (because having both DOTNET and
-	// ASPNETCORE versions is already confusing enough). For now, we'll use `ASPIRE_ENVIRONMENT` to seed the initial values of
-	// `DOTNET_ENVIRONMENT`, but allow them to be overriden at environment construction time.
+	// ASPNETCORE versions is already confusing enough). For now, we'll use `ASPIRE_ENVIRONMENT` to seed the initial values
+	// of `DOTNET_ENVIRONMENT`, but allow them to be overriden at environment construction time.
 	//
 	// We only retain `DOTNET_ENVIRONMENT` in the .env file.
 	dotnetEnv := newEnv.Properties["ASPIRE_ENVIRONMENT"]

cli/azd/pkg/auth/manager.go

@@ -300,7 +300,8 @@ func (m *Manager) CredentialForCurrentUser(
 						return nil, err
 					}
 
-					return newAzdCredential(&msalPublicClientAdapter{client: &clientWithNewTenant}, &accounts[i], m.cloud), nil
+					return newAzdCredential(
+						&msalPublicClientAdapter{client: &clientWithNewTenant}, &accounts[i], m.cloud), nil
 				}
 			}
 		}

cli/azd/pkg/pipeline/github_provider.go

@@ -652,7 +652,8 @@ func (p *GitHubCiProvider) configurePipeline(
 			p.console.MessageUxItem(ctx, &ux.MultilineMessage{
 				Lines: []string{
 					"",
-					"GitHub Action secrets are now configured. You can view GitHub action secrets that were created at this link:",
+					"GitHub Action secrets are now configured. You can view GitHub action secrets that were " +
+						"created at this link:",
 					output.WithLinkFormat("https://github.com/%s/settings/secrets/actions", repoSlug),
 					""},
 			})

cli/azd/pkg/pipeline/pipeline_manager.go

@@ -458,7 +458,8 @@ func (pm *PipelineManager) Configure(ctx context.Context, projectName string) (r
 	} else {
 		pm.console.Message(ctx,
 			fmt.Sprintf(
-				"To fully enable pipeline you need to push this repo to the upstream using 'git push --set-upstream %s %s'.\n",
+				"To fully enable pipeline you need to push this repo to the upstream "+
+					"using 'git push --set-upstream %s %s'.\n",
 				pm.args.PipelineRemoteName,
 				gitRepoInfo.branch))
 	}
@@ -881,7 +882,8 @@ func (pm *PipelineManager) promptForCiFiles(ctx context.Context, pipelineProvide
 	// Confirm with the user before adding the file
 	pm.console.Message(ctx, "")
 	pm.console.Message(ctx,
-		fmt.Sprintf("The default %s file, which contains a basic workflow to help you get started, is missing from your project.",
+		fmt.Sprintf(
+			"The default %s file, which contains a basic workflow to help you get started, is missing from your project.",
 			output.WithHighLightFormat("azure-dev.yml")))
 	pm.console.Message(ctx, "")
 

cli/azd/pkg/pipeline/pipeline_manager_test.go

@@ -92,7 +92,8 @@ func Test_PipelineManager_Initialize(t *testing.T) {
 
 		manager, err := createPipelineManager(t, mockContext, azdContext, nil, nil)
 		assert.Nil(t, manager)
-		assert.EqualError(t, err, fmt.Sprintf("%s provider selected, but %s is empty. Please add pipeline files and try again.",
+		assert.EqualError(t, err, fmt.Sprintf(
+			"%s provider selected, but %s is empty. Please add pipeline files and try again.",
 			azdoDisplayName, azdoPipelinesDirectory))
 	})
 	t.Run("no files - azdo selected", func(t *testing.T) {
@@ -129,7 +130,8 @@ func Test_PipelineManager_Initialize(t *testing.T) {
 
 		manager, err := createPipelineManager(t, mockContext, azdContext, env, nil)
 		assert.Nil(t, manager)
-		assert.EqualError(t, err, fmt.Sprintf("%s provider selected, but %s is empty. Please add pipeline files and try again.",
+		assert.EqualError(t, err, fmt.Sprintf(
+			"%s provider selected, but %s is empty. Please add pipeline files and try again.",
 			azdoDisplayName, azdoPipelinesDirectory))
 	})
 	t.Run("from persisted data azdo", func(t *testing.T) {

cli/azd/pkg/tools/azcli/azcli_linuxwebapp_test.go

@@ -99,7 +99,8 @@ func registerIsLinuxWebAppMocks(mockContext *mocks.MockContext, ran *bool) {
 	mockContext.HttpClient.When(func(request *http.Request) bool {
 		return request.Method == http.MethodGet &&
 			strings.Contains(request.URL.Path,
-				"/subscriptions/SUBSCRIPTION_ID/resourceGroups/RESOURCE_GROUP_ID/providers/Microsoft.Web/sites/LINUX_WEB_APP_NAME")
+				"/subscriptions/SUBSCRIPTION_ID/resourceGroups/RESOURCE_GROUP_ID/providers/Microsoft.Web/sites/"+
+					"LINUX_WEB_APP_NAME")
 	}).RespondFn(func(request *http.Request) (*http.Response, error) {
 		*ran = true
 		response := armappservice.WebAppsClientGetResponse{
@@ -130,7 +131,8 @@ func registerLogicAppMocks(mockContext *mocks.MockContext, ran *bool) {
 	mockContext.HttpClient.When(func(request *http.Request) bool {
 		return request.Method == http.MethodGet &&
 			strings.Contains(request.URL.Path,
-				"/subscriptions/SUBSCRIPTION_ID/resourceGroups/RESOURCE_GROUP_ID/providers/Microsoft.Web/sites/WINDOWS_LOGIC_APP_NAME")
+				"/subscriptions/SUBSCRIPTION_ID/resourceGroups/RESOURCE_GROUP_ID/providers/Microsoft.Web/sites/"+
+					"WINDOWS_LOGIC_APP_NAME")
 	}).RespondFn(func(request *http.Request) (*http.Response, error) {
 		*ran = true
 		response := armappservice.WebAppsClientGetResponse{

cli/azd/pkg/tools/azcli/webapp.go

@@ -140,7 +140,8 @@ func (cli *azCli) DeployAppServiceZip(
 
 	// Deployment Status API only support linux web app for now
 	if isLinuxWebApp(app) {
-		if err := client.DeployTrackStatus(ctx, deployZipFile, subscriptionId, resourceGroup, appName, progressLog); err != nil {
+		if err := client.DeployTrackStatus(
+			ctx, deployZipFile, subscriptionId, resourceGroup, appName, progressLog); err != nil {
 			if !resumeDeployment(err, progressLog) {
 				return nil, err
 			}

cli/azd/pkg/tools/dotnet/dotnet.go

@@ -141,8 +141,8 @@ func (cli *dotNetCli) PublishAppHostManifest(
 ) error {
 	// TODO(ellismg): Before we GA manifest support, we should remove this debug tool, but being able to control what
 	// manifest is used is helpful, while the manifest/generator is still being built.  So if
-	// `AZD_DEBUG_DOTNET_APPHOST_USE_FIXED_MANIFEST` is set, then we will expect to find apphost-manifest.json SxS with the host
-	// project, and we just use that instead.
+	// `AZD_DEBUG_DOTNET_APPHOST_USE_FIXED_MANIFEST` is set, then we will expect to find apphost-manifest.json SxS with the
+	// host project, and we just use that instead.
 	if enabled, err := strconv.ParseBool(os.Getenv("AZD_DEBUG_DOTNET_APPHOST_USE_FIXED_MANIFEST")); err == nil && enabled {
 		m, err := os.ReadFile(filepath.Join(filepath.Dir(hostProject), "apphost-manifest.json"))
 		if err != nil {
@@ -291,7 +291,7 @@ func (cli *dotNetCli) SetSecrets(ctx context.Context, secrets map[string]string,
 	}
 
 	// dotnet user-secrets now support setting multiple values at once
-	//https://learn.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-7.0&tabs=windows#set-multiple-secrets
+	// learn.microsoft.com/aspnet/core/security/app-secrets?view=aspnetcore-7.0&tabs=windows#set-multiple-secrets
 	runArgs := newDotNetRunArgs("user-secrets", "set", "--project", project).
 		WithStdIn(strings.NewReader(string(secretsJson)))
 

cli/azd/test/mocks/mockai/deployments.go

@@ -103,7 +103,8 @@ func RegisterListOnlineDeployment(
 		}
 
 		response := armmachinelearning.OnlineDeploymentsClientListResponse{
-			OnlineDeploymentTrackedResourceArmPaginatedResult: armmachinelearning.OnlineDeploymentTrackedResourceArmPaginatedResult{
+			OnlineDeploymentTrackedResourceArmPaginatedResult: armmachinelearning.
+				OnlineDeploymentTrackedResourceArmPaginatedResult{
 				Value: deployments,
 			},
 		}