Fixed linter errors

Author: xdoubleu

pkg/database/postgres/pgx_sync_tx.go

@@ -167,7 +167,10 @@ func (tx *PgxSyncTx) Begin(ctx context.Context) (pgx.Tx, error) {
 }
 
 // BeginTx is used to wrap [pgx.Tx.BeginTx] in a [database.SyncTx].
-func (tx *PgxSyncTx) BeginTx(ctx context.Context, txOptions pgx.TxOptions) (pgx.Tx, error) {
+func (tx *PgxSyncTx) BeginTx(
+	ctx context.Context,
+	txOptions pgx.TxOptions,
+) (pgx.Tx, error) {
 	return database.WrapInSyncTx(
 		ctx,
 		tx.syncTx,

pkg/database/postgres/spandb.go

@@ -79,7 +79,10 @@ func (db *SpanDB) Begin(ctx context.Context) (pgx.Tx, error) {
 
 // BeginTx doesn't wrap BeginTx in a [sentry.Span] as
 // this makes little sense for starting a transaction.
-func (db *SpanDB) BeginTx(ctx context.Context, txOptions pgx.TxOptions) (pgx.Tx, error) {
+func (db *SpanDB) BeginTx(
+	ctx context.Context,
+	txOptions pgx.TxOptions,
+) (pgx.Tx, error) {
 	return db.DB.BeginTx(ctx, txOptions)
 }
 

pkg/database/span.go

@@ -6,6 +6,7 @@ import (
 	"github.com/getsentry/sentry-go"
 )
 
+// StartSpan is used to start a [sentry.Span].
 func StartSpan(ctx context.Context, dbName string, sql string) *sentry.Span {
 	span := sentry.StartSpan(ctx, "db.query", sentry.WithDescription(sql))
 	span.SetData("db.system", dbName)

pkg/parse/parser_funcs.go

@@ -14,7 +14,7 @@ type ParserFunc[T any] func(paramType string, paramName string, value string) (T
 
 // String is used to parse a parameter as string value.
 // As all parameters are string by default this returns the original value.
-func String(paramType string, paramName string, value string) (string, error) {
+func String(_ string, _ string, value string) (string, error) {
 	return value, nil
 }
 

pkg/time/main.go

@@ -29,9 +29,9 @@ func EndOfDay(dateTime time.Time) time.Time {
 	return output
 }
 
-// NowTimeZoneIndependent returns the provided time in the
+// LocationIndependentTime returns the provided time in the
 // provided time zone but forces the time zone to UTC.
-func TimeZoneIndependentTime(t time.Time, locationTimeZone string) time.Time {
+func LocationIndependentTime(t time.Time, locationTimeZone string) time.Time {
 	timeZone, _ := time.LoadLocation(locationTimeZone)
 	utcTimeZone, _ := time.LoadLocation("UTC")
 	t = t.In(timeZone)

pkg/time/main_test.go

@@ -34,11 +34,11 @@ func TestEndOfDay(t *testing.T) {
 	assert.Equal(t, endOfDay, timetools.EndOfDay(now))
 }
 
-func TestTimeZoneIndependentTime(t *testing.T) {
+func TestLocationIndependentTime(t *testing.T) {
 	now := time.Now()
 
 	utcTimeZone, _ := time.LoadLocation("UTC")
-	result := timetools.TimeZoneIndependentTime(now, now.Location().String())
+	result := timetools.LocationIndependentTime(now, now.Location().String())
 
 	expected := time.Date(
 		now.Year(),

pkg/validate/main.go

@@ -23,6 +23,7 @@ func (v *Validator) Valid() bool {
 	return len(v.errors) == 0
 }
 
+// Errors returns the [Validator] errors.
 func (v *Validator) Errors() map[string]string {
 	return v.errors
 }
@@ -41,7 +42,14 @@ func Check[T any](v *Validator, key string, value T, validatorFunc ValidatorFunc
 	}
 }
 
-func CheckOptional[T any](v *Validator, key string, value *T, validatorFunc ValidatorFunc[T]) {
+// CheckOptional checks if value passes the validatorFunc, when the value is provided.
+// The provided key is used for creating the errors map of the [Validator].
+func CheckOptional[T any](
+	v *Validator,
+	key string,
+	value *T,
+	validatorFunc ValidatorFunc[T],
+) {
 	if value == nil {
 		return
 	}

pkg/validate/validator_test.go

@@ -1,3 +1,4 @@
+//nolint:exhaustruct //on purpose
 package validate_test
 
 import (
@@ -42,7 +43,12 @@ func (ts *TestStruct) Validate() (bool, map[string]string) {
 
 	validate.Check(v, "tzVal", ts.TzVal, validate.IsValidTimeZone)
 
-	validate.CheckOptional(v, "optStrVal", ts.OptStrVal, validate.IsInSlice([]string{"allowed"}))
+	validate.CheckOptional(
+		v,
+		"optStrVal",
+		ts.OptStrVal,
+		validate.IsInSlice([]string{"allowed"}),
+	)
 
 	return v.Valid(), v.Errors()
 }