chore: edit linter settings

Author: EpicStep

.golangci.yml

@@ -12,6 +12,27 @@ linters:
     - paralleltest
     - revive
   settings:
+    revive:
+      enable-all-rules: true
+      rules:
+        - name: add-constant
+          disabled: true
+        - name: var-naming
+          disabled: true
+        - name: line-length-limit
+          disabled: true
+        - name: function-length
+          disabled: true
+        - name: cognitive-complexity
+          arguments: [ 30 ]
+        - name: cyclomatic
+          disabled: true
+        - name: import-alias-naming
+          disabled: true
+        - name: unused-receiver
+          disabled: true
+        - name: unhandled-error
+          arguments: [ "fmt.Println" ]
     goheader:
       template: |-
         Copyright {{ YEAR }} LLC "Ozon Technologies".

internal/utils/database/database.go

@@ -16,26 +16,26 @@ import (
 var errInvalidArguments = errors.New("invalid arguments")
 
 // CreateTableAs ...
-func CreateTableAs(ctx context.Context, shard database.Shard, database, srcTable, dstTable string) error {
-	if sqlUtils.ValidateEntityName(database) != nil || sqlUtils.ValidateEntityName(srcTable) != nil || sqlUtils.ValidateEntityName(dstTable) != nil {
+func CreateTableAs(ctx context.Context, shard database.Shard, databaseName, srcTableName, dstTableName string) error {
+	if sqlUtils.ValidateEntityName(databaseName) != nil || sqlUtils.ValidateEntityName(srcTableName) != nil || sqlUtils.ValidateEntityName(dstTableName) != nil {
 		return errInvalidArguments
 	}
 
-	if err := shard.Exec(ctx, fmt.Sprintf("CREATE TABLE %s AS %s", sqlUtils.QuotedDatabaseEntity(database, dstTable), sqlUtils.QuotedDatabaseEntity(database, srcTable))); err != nil {
-		return fmt.Errorf("failed to create table %s as %s in %s: %w", dstTable, srcTable, database, err)
+	if err := shard.Exec(ctx, fmt.Sprintf("CREATE TABLE %s AS %s", sqlUtils.QuotedDatabaseEntity(databaseName, dstTableName), sqlUtils.QuotedDatabaseEntity(databaseName, srcTableName))); err != nil {
+		return fmt.Errorf("failed to create table %s as %s in %s: %w", dstTableName, srcTableName, databaseName, err)
 	}
 
 	return nil
 }
 
 // DropTable ...
-func DropTable(ctx context.Context, shard database.Shard, database, table string) error {
-	if sqlUtils.ValidateEntityName(database) != nil || sqlUtils.ValidateEntityName(table) != nil {
+func DropTable(ctx context.Context, shard database.Shard, databaseName, tableName string) error {
+	if sqlUtils.ValidateEntityName(databaseName) != nil || sqlUtils.ValidateEntityName(tableName) != nil {
 		return errInvalidArguments
 	}
 
-	if err := shard.Exec(ctx, "DROP TABLE "+sqlUtils.QuotedDatabaseEntity(database, table)); err != nil {
-		return fmt.Errorf("failed to drop table %s in %s: %w", table, database, err)
+	if err := shard.Exec(ctx, "DROP TABLE "+sqlUtils.QuotedDatabaseEntity(databaseName, tableName)); err != nil {
+		return fmt.Errorf("failed to drop table %s in %s: %w", tableName, databaseName, err)
 	}
 
 	return nil

internal/utils/time/time.go

@@ -2,7 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 // Package time is a utils for working with time.
-package time //revive:disable-line:var-naming
+package time
 
 import "time"
 

internal/utils/time/time_test.go

@@ -1,7 +1,7 @@
 // Copyright 2025 LLC "Ozon Technologies".
 // SPDX-License-Identifier: Apache-2.0
 
-package time //revive:disable-line:var-naming
+package time
 
 import (
 	"testing"

pkg/database/error_wrappers/clickhouse_go/wrap.go

@@ -2,7 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 
 // Package clickhouse_go implements error wrapper for 'clickhouse-go'.
-package clickhouse_go //revive:disable:var-naming
+package clickhouse_go
 
 import (
 	"errors"

pkg/database/error_wrappers/clickhouse_go/wrap_test.go

@@ -1,7 +1,7 @@
 // Copyright 2025 LLC "Ozon Technologies".
 // SPDX-License-Identifier: Apache-2.0
 
-package clickhouse_go //revive:disable:var-naming
+package clickhouse_go
 
 import (
 	"errors"

pkg/database/error_wrappers/common/wrap_test.go

@@ -1,7 +1,7 @@
 // Copyright 2025 LLC "Ozon Technologies".
 // SPDX-License-Identifier: Apache-2.0
 
-package common //revive:disable:var-naming
+package common
 
 import (
 	"context"

pkg/rollup/partitions.go

@@ -13,13 +13,13 @@ import (
 	"github.com/ozontech/ch-rollup/pkg/database"
 )
 
-func getPartitionsOnShard(ctx context.Context, shard database.Shard, database, table string) ([]string, error) {
+func getPartitionsOnShard(ctx context.Context, shard database.Shard, databaseName, tableName string) ([]string, error) {
 	sb := sqlbuilder.NewSelectBuilder().From("system.parts")
 
 	sb.Select("partition")
 	sb.Where(
-		sb.Equal("database", database),
-		sb.Equal("table", table),
+		sb.Equal("database", databaseName),
+		sb.Equal("table", tableName),
 		sb.Equal("active", 1),
 	)
 	sb.GroupBy("partition")
@@ -56,14 +56,14 @@ func getPartitionsOnShard(ctx context.Context, shard database.Shard, database, t
 
 // replacePartitionsOnShard replaces partitions on shard.
 // Arguments must be sanitized.
-func replacePartitionsOnShard(ctx context.Context, shard database.Shard, database, from, to string, partitions []string) error {
+func replacePartitionsOnShard(ctx context.Context, shard database.Shard, databaseName, from, to string, partitions []string) error {
 	// TODO: generate multistatement query.
 	for _, partition := range partitions {
 		b := sqlbuilder.Build(
 			"ALTER TABLE $? REPLACE PARTITION $? FROM $?",
-			sqlbuilder.Raw(sqlUtils.QuotedDatabaseEntity(database, to)),
+			sqlbuilder.Raw(sqlUtils.QuotedDatabaseEntity(databaseName, to)),
 			partition,
-			sqlbuilder.Raw(sqlUtils.QuotedDatabaseEntity(database, from)),
+			sqlbuilder.Raw(sqlUtils.QuotedDatabaseEntity(databaseName, from)),
 		)
 
 		sql, args := b.BuildWithFlavor(sqlbuilder.ClickHouse)

pkg/types/types_test.go

@@ -1,7 +1,7 @@
 // Copyright 2025 LLC "Ozon Technologies".
 // SPDX-License-Identifier: Apache-2.0
 
-package types //revive:disable:var-naming
+package types
 
 import (
 	"testing"