🌱 (chore): remove unnecessary trailing newlines and redundant blank lines (#4664)

chore: remove unnecessary trailing newlines and redundant blank lines

Cleaned up redundant blank lines and unnecessary trailing newlines across various files,
including test suites, scaffolds, plugin templates, and documentation scripts. This improves
code consistency and reduces visual noise, particularly in generated and test-related files.

Author: kersten

hack/docs/internal/cronjob-tutorial/controller_implementation.go

@@ -536,6 +536,7 @@ var (
 	apiGVStr    = batchv1.GroupVersion.String()
 )
 `
+
 const controllerSetupWithManager = `
 	// set up a real clock, since we're not in a test
 	if r.Clock == nil {

hack/docs/internal/cronjob-tutorial/e2e_implementation.go

@@ -47,6 +47,7 @@ if !isPrometheusOperatorAlreadyInstalled {
 	_, _ = fmt.Fprintf(GinkgoWriter, "WARNING: Prometheus Operator is already installed. Skipping installation...\n")
 }
 `
+
 const serviceMonitorE2e = `
 
 By("validating that the ServiceMonitor for Prometheus is applied in the namespace")

hack/docs/internal/cronjob-tutorial/generate_cronjob.go

@@ -387,7 +387,6 @@ manifests: controller-gen ## Generate WebhookConfiguration, ClusterRole and Cust
 `
 	err := pluginutil.ReplaceInFile(filepath.Join(sp.ctx.Dir, "Makefile"), originalManifestTarget, changedManifestTarget)
 	hackutils.CheckError("updating makefile to use maxDescLen=0 in make manifest target", err)
-
 }
 
 func (sp *Sample) updateWebhookTests() {

hack/docs/internal/cronjob-tutorial/webhook_implementation.go

@@ -313,6 +313,7 @@ const webhookTestsVars = `var (
 		validator CronJobCustomValidator
 		defaulter CronJobCustomDefaulter
 	)`
+
 const webhookTestsConstants = `	var (
 		obj       *batchv1.CronJob
 		oldObj    *batchv1.CronJob
@@ -322,6 +323,7 @@ const webhookTestsConstants = `	var (
 
 	const validCronJobName = "valid-cronjob-name"
 	const schedule = "*/5 * * * *"`
+
 const webhookTestsBeforeEachOriginal = `obj = &batchv1.CronJob{}
 		oldObj = &batchv1.CronJob{}
 		validator = CronJobCustomValidator{}

hack/docs/internal/getting-started/generate_getting_started.go

@@ -430,6 +430,7 @@ const controllerReconcileImplementation = `// Fetch the Memcached instance
 		log.Error(err, "Failed to update Memcached status")
 		return ctrl.Result{}, err
 	}`
+
 const controllerDeploymentFunc = `// deploymentForMemcached returns a Memcached Deployment object
 func (r *MemcachedReconciler) deploymentForMemcached(
 	memcached *cachev1alpha1.Memcached) (*appsv1.Deployment, error) {

hack/docs/internal/multiversion-tutorial/generate_multiversion.go

@@ -658,7 +658,6 @@ CronJob controller's `+"`SetupWithManager`"+` method.
 		`// +kubebuilder:docs-gen:collapse=existing setup`,
 	)
 	hackutils.CheckError("replace +kubebuilder:docs-gen:collapse=old stuff main.go", err)
-
 }
 
 func (sp *Sample) updateAPIV2() {

pkg/cli/cli_test.go

@@ -467,7 +467,6 @@ plugins:
 				printed, _ := io.ReadAll(r)
 				Expect(string(printed)).To(Equal(
 					fmt.Sprintf("%s\n", version)))
-
 			})
 		})
 

pkg/cli/completion_test.go

@@ -78,6 +78,5 @@ var _ = Describe("Completion", func() {
 			Expect(cmd.Short).NotTo(Equal(""))
 			Expect(cmd.Short).To(ContainSubstring("Load powershell completions"))
 		})
-
 	})
 })

pkg/plugin/bundle.go

@@ -60,7 +60,6 @@ func WithDeprecationMessage(msg string) BundleOption {
 	return func(opts *bundle) {
 		opts.deprecateWarning = msg
 	}
-
 }
 
 // NewBundleWithOptions creates a new Bundle with the provided BundleOptions.

pkg/plugins/common/kustomize/v2/create.go

@@ -52,7 +52,6 @@ func (p *createSubcommand) configure() (err error) {
 		if p.force, err = strconv.ParseBool(forceFlag.Value.String()); err != nil {
 			return err
 		}
-
 	}
 	return nil
 }