Fix styling

neoxelox · Jan 29, 2024 · 4c8bfa2 · 4c8bfa2
1 parent b41f5dc
commit 4c8bfa2
Show file tree

Hide file tree

Showing 11 changed files with 27 additions and 12 deletions.
diff --git a/.golangci.yaml b/.golangci.yaml
@@ -148,6 +148,11 @@ linters-settings:
     excludes:
       - G114
 
+  # Checks to simplify the code.
+  gosimple:
+    # Sxxxx checks in https://staticcheck.io/docs/configuration/options/#checks
+    checks: ["all", "-S1016"]
+
   # Checks for suspicious constructs.
   govet:
     # Report about shadowed variables.

diff --git a/cmd/api/api.go b/cmd/api/api.go
@@ -214,7 +214,8 @@ func NewAPI(ctx context.Context, config config.Config) (*API, error) {
 	healthEndpoints := util.NewHealthEndpoints(observer, database, cache, config)
 	fileEndpoints := util.NewFileEndpoints(observer, config)
 
-	exampleEndpoints := example.NewExampleEndpoints(observer, localizer, exampleGetter, exampleCreator, exampleDeleter, config)
+	exampleEndpoints := example.NewExampleEndpoints(
+		observer, localizer, exampleGetter, exampleCreator, exampleDeleter, config)
 
 	/* MIDDLEWARES */
 

diff --git a/cmd/cli/cli.go b/cmd/cli/cli.go
@@ -163,11 +163,13 @@ func NewCLI(ctx context.Context, config config.Config) (*CLI, error) {
 
 	/* REGISTRATIONS */
 
-	runner.Register(util.DatabaseCommandsRollback, databaseCommands.Rollback, util.DatabaseCommandsRollbackArgs{}, "rollback migrations")
+	runner.Register(util.DatabaseCommandsRollback, databaseCommands.Rollback,
+		util.DatabaseCommandsRollbackArgs{}, "rollback migrations")
 
 	runner.Use(exampleMiddlewares.HandleCommand)
 
-	runner.Register(example.ExampleCommandsForceOnboarding, exampleCommands.ForceOnboarding, example.ExampleCommandsForceOnboardingArgs{}, "force onboarding to an example")
+	runner.Register(example.ExampleCommandsForceOnboarding, exampleCommands.ForceOnboarding,
+		example.ExampleCommandsForceOnboardingArgs{}, "force onboarding to an example")
 
 	return &CLI{
 		Run: func(ctx context.Context) error {

diff --git a/migrations/0002_example.up.sql b/migrations/0002_example.up.sql
@@ -1,7 +1,7 @@
 CREATE TABLE IF NOT EXISTS "example" (
     "id" VARCHAR(20) PRIMARY KEY,
     "name" VARCHAR(100) NOT NULL,
-    "age" INTEGER NOT NULL,
+    "age" BIGINT NOT NULL,
     "country" VARCHAR(2) NOT NULL,
     "role" VARCHAR(50) NOT NULL,
     "settings" JSONB NOT NULL,

diff --git a/pkg/example/creator.go b/pkg/example/creator.go
@@ -22,7 +22,8 @@ type ExampleCreator struct {
 	enqueuer          *kit.Enqueuer
 }
 
-func NewExampleCreator(observer *kit.Observer, exampleService *ExampleService, exampleRepository *ExampleRepository, enqueuer *kit.Enqueuer, config config.Config) *ExampleCreator {
+func NewExampleCreator(observer *kit.Observer, exampleService *ExampleService, exampleRepository *ExampleRepository,
+	enqueuer *kit.Enqueuer, config config.Config) *ExampleCreator {
 	return &ExampleCreator{
 		config:            config,
 		observer:          observer,

diff --git a/pkg/example/deleter.go b/pkg/example/deleter.go
@@ -15,7 +15,8 @@ type ExampleDeleter struct {
 	exampleRepository *ExampleRepository
 }
 
-func NewExampleDeleter(observer *kit.Observer, database *kit.Database, exampleRepository *ExampleRepository, config config.Config) *ExampleDeleter {
+func NewExampleDeleter(observer *kit.Observer, database *kit.Database, exampleRepository *ExampleRepository,
+	config config.Config) *ExampleDeleter {
 	return &ExampleDeleter{
 		config:            config,
 		observer:          observer,

diff --git a/pkg/example/endpoints.go b/pkg/example/endpoints.go
@@ -23,7 +23,8 @@ type ExampleEndpoints struct {
 	exampleDeleter *ExampleDeleter
 }
 
-func NewExampleEndpoints(observer *kit.Observer, localizer *kit.Localizer, exampleGetter *ExampleGetter, exampleCreator *ExampleCreator, exampleDeleter *ExampleDeleter, config config.Config) *ExampleEndpoints {
+func NewExampleEndpoints(observer *kit.Observer, localizer *kit.Localizer, exampleGetter *ExampleGetter,
+	exampleCreator *ExampleCreator, exampleDeleter *ExampleDeleter, config config.Config) *ExampleEndpoints {
 	return &ExampleEndpoints{
 		config:         config,
 		observer:       observer,

diff --git a/pkg/example/getter.go b/pkg/example/getter.go
@@ -21,7 +21,8 @@ type ExampleGetter struct {
 	exampleRepository *ExampleRepository
 }
 
-func NewExampleGetter(observer *kit.Observer, cache *kit.Cache, exampleRepository *ExampleRepository, config config.Config) *ExampleGetter {
+func NewExampleGetter(observer *kit.Observer, cache *kit.Cache, exampleRepository *ExampleRepository,
+	config config.Config) *ExampleGetter {
 	return &ExampleGetter{
 		config:            config,
 		observer:          observer,

diff --git a/pkg/example/services.go b/pkg/example/services.go
@@ -58,8 +58,10 @@ type ExampleServiceGetCountryResult struct {
 	Country string
 }
 
-func (self *ExampleService) GetCountry(ctx context.Context, params ExampleServiceGetCountryParams) (*ExampleServiceGetCountryResult, error) {
-	response, err := self.client.Request(ctx, "GET", fmt.Sprintf("%s/?name=%s", self.config.ExampleService.BaseURL, params.Name), nil)
+func (self *ExampleService) GetCountry(ctx context.Context,
+	params ExampleServiceGetCountryParams) (*ExampleServiceGetCountryResult, error) {
+	response, err := self.client.Request(
+		ctx, "GET", fmt.Sprintf("%s/?name=%s", self.config.ExampleService.BaseURL, params.Name), nil)
 	if err != nil {
 		if kit.ErrHTTPClientTimedOut.Is(err) {
 			return nil, ErrExampleServiceTimedOut.Raise().Cause(err)

diff --git a/pkg/example/tasks.go b/pkg/example/tasks.go
@@ -65,7 +65,7 @@ func (self *ExampleTasks) MakeOnboarding(ctx context.Context, task *asynq.Task)
 	return nil
 }
 
-func (self *ExampleTasks) Reconcile(ctx context.Context, task *asynq.Task) error {
+func (self *ExampleTasks) Reconcile(ctx context.Context, _ *asynq.Task) error {
 	self.observer.Info(ctx, "reconciling examples")
 
 	return nil

diff --git a/pkg/util/endpoints.go b/pkg/util/endpoints.go
@@ -23,7 +23,8 @@ type HealthEndpoints struct {
 	cache    *kit.Cache
 }
 
-func NewHealthEndpoints(observer *kit.Observer, database *kit.Database, cache *kit.Cache, config config.Config) *HealthEndpoints {
+func NewHealthEndpoints(observer *kit.Observer, database *kit.Database, cache *kit.Cache,
+	config config.Config) *HealthEndpoints {
 	return &HealthEndpoints{
 		config:   config,
 		observer: observer,