WIP: refactor: refactor funcs with generics #1123
reviewdog [golangci] report
reported by reviewdog 🐶
Findings (20)
internal/template/method.go|30 col 1| exported: comment on exported const GetInstanceMethod should be of the form "GetInstanceMethod ..." (revive)
field/number.go|8 col 6| exported: exported type Int should have comment or be unexported (revive)
field/number.go|10 col 6| exported: exported type Int8 should have comment or be unexported (revive)
field/number.go|12 col 6| exported: exported type Int16 should have comment or be unexported (revive)
field/number.go|14 col 6| exported: exported type Int32 should have comment or be unexported (revive)
field/number.go|16 col 6| exported: exported type Int64 should have comment or be unexported (revive)
field/number.go|18 col 6| exported: exported type Uint should have comment or be unexported (revive)
field/number.go|20 col 6| exported: exported type Uint8 should have comment or be unexported (revive)
field/number.go|22 col 6| exported: exported type Uint16 should have comment or be unexported (revive)
field/number.go|24 col 6| exported: exported type Uint32 should have comment or be unexported (revive)
field/number.go|26 col 6| exported: exported type Uint64 should have comment or be unexported (revive)
field/number.go|28 col 6| exported: exported type Float32 should have comment or be unexported (revive)
field/number.go|30 col 6| exported: exported type Float64 should have comment or be unexported (revive)
field/time.go|81 col 1| exported: exported method Time.Year should have comment or be unexported (revive)
field/export.go|40 col 53| unexported-return: exported func NewField returns unexported type field.genericsField[database/sql/driver.Valuer], which can be annoying to use (revive)
generics.go|200 col 9| indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
generics.go|209 col 9| indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
generics.go|218 col 9| indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
generics.go|13 col 6| exported: exported type IGenericsDo should have comment or be unexported (revive)
generics.go|75 col 6| exported: exported type GenericsDo should have comment or be unexported (revive)
Filtered Findings (46)
field/expr.go|127 col 23| Error return value of builder.WriteString is not checked (errcheck)
field/serializer.go|22 col 13| Error return value of db.AddError is not checked (errcheck)
do.go|263 col 18| Error return value of stmt.WriteByte is not checked (errcheck)
do.go|294 col 18| Error return value of stmt.WriteByte is not checked (errcheck)
internal/template/method.go|1 col 1| package-comments: should have a package comment (revive)
internal/utils/common.go|1 col 1| package-comments: should have a package comment (revive)
field/tag.go|11 col 2| exported: exported const TagKeyGorm should have comment (or a comment on this block) or be unexported (revive)
field/tag.go|14 col 2| exported: comment on exported const TagKeyGormColumn should be of the form "TagKeyGormColumn ..." (revive)
field/tag.go|43 col 6| exported: exported type TagBuilder should have comment or be unexported (revive)
field/tag.go|47 col 6| exported: exported type Tag should have comment or be unexported (revive)
field/tag.go|49 col 1| exported: exported method Tag.Set should have comment or be unexported (revive)
field/tag.go|54 col 1| exported: exported method Tag.Remove should have comment or be unexported (revive)
field/tag.go|59 col 1| exported: exported method Tag.Build should have comment or be unexported (revive)
field/tag.go|87 col 6| exported: exported type GormTag should have comment or be unexported (revive)
field/tag.go|89 col 1| exported: exported method GormTag.Append should have comment or be unexported (revive)
field/tag.go|98 col 1| exported: exported method GormTag.Set should have comment or be unexported (revive)
field/tag.go|103 col 1| exported: exported method GormTag.Remove should have comment or be unexported (revive)
field/tag.go|108 col 1| exported: exported method GormTag.Build should have comment or be unexported (revive)
field/tag.go|12 col 2| var-naming: const TagKeyJson should be TagKeyJSON (revive)
field/expr.go|57 col 6| exported: exported type IColumnName should have comment or be unexported (revive)
field/assign_attr.go|13 col 6| exported: exported type IValues should have comment or be unexported (revive)
field/association.go|240 col 1| exported: exported method RelateConfig.GetTag should have comment or be unexported (revive)
field/export.go|339 col 118| unused-parameter: parameter 'relations' seems to be unused, consider removing or renaming it as _ (revive)
field/assign_attr.go|115 col 1| exported: exported function Attrs should have comment or be unexported (revive)
field/serializer.go|21 col 33| context-keys-type: should not use basic type string as key in context.WithValue (revive)
helper/clause.go|1 col 1| package-comments: should have a package comment (revive)
internal/parser/method.go|12 col 1| exported: exported function DefaultMethodTableName should have comment or be unexported (revive)
internal/generate/table.go|74 col 37| unused-parameter: parameter 'schemaName' seems to be unused, consider removing or renaming it as _ (revive)
internal/generate/table.go|86 col 35| unused-parameter: parameter 'schemaName' seems to be unused, consider removing or renaming it as _ (revive)
internal/generate/query.go|17 col 6| exported: exported type FieldParser should have comment or be unexported (revive)
do_options.go|21 col 36| unused-parameter: parameter 'db' seems to be unused, consider removing or renaming it as _ (revive)
do_options.go|9 col 6| exported: exported type DOConfig should have comment or be unexported (revive)
field_options.go|23 col 2| exported: exported var FieldModify should have comment or be unexported (revive)
field_options.go|266 col 2| exported: exported var DefaultMethodTableWithNamer should have comment or be unexported (revive)
internal/generate/section.go|563 col 108| (*Section).checkSQLVar - result err is always nil (unparam)
field/expr.go|422 col 15| func expr.field is unused (unused)
internal/model/options.go|10 col 5| var defaultSchemaNameOpt is unused (unused)
internal/generate/interface.go|446 col 27| func (*InterfaceMethod).checkSQLVarByParams is unused (unused)
internal/generate/interface.go|487 col 27| func (*InterfaceMethod).isParamExist is unused (unused)
internal/generate/section.go|694 col 2| field suffix is unused (unused)
field_options.go|20 col 5| var ns is unused (unused)
generator.go|302 col 4| shadow: declaration of "err" shadows declaration at line 286 (govet)
generator.go|489 col 4| shadow: declaration of "err" shadows declaration at line 469 (govet)
internal/model/base.go|204 col 10| SA1019: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead. (staticcheck)
internal/model/base.go|206 col 10| SA1019: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead. (staticcheck)
internal/model/base.go|208 col 10| SA1019: strings.Title has been deprecated since Go 1.18 and an alternative has been available since Go 1.0: The rule Title uses for word boundaries does not handle Unicode punctuation properly. Use golang.org/x/text/cases instead. (staticcheck)
Annotations
Check failure on line 30 in internal/template/method.go
github-actions / golangci
[golangci] internal/template/method.go#L30
exported: comment on exported const GetInstanceMethod should be of the form "GetInstanceMethod ..." (revive)
Raw output
internal/template/method.go:30:1: exported: comment on exported const GetInstanceMethod should be of the form "GetInstanceMethod ..." (revive)
// GetInstance method
^
Check failure on line 8 in field/number.go
github-actions / golangci
[golangci] field/number.go#L8
exported: exported type Int should have comment or be unexported (revive)
Raw output
field/number.go:8:6: exported: exported type Int should have comment or be unexported (revive)
type Int = Number[int]
^
Check failure on line 10 in field/number.go
github-actions / golangci
[golangci] field/number.go#L10
exported: exported type Int8 should have comment or be unexported (revive)
Raw output
field/number.go:10:6: exported: exported type Int8 should have comment or be unexported (revive)
type Int8 = Number[int8]
^
Check failure on line 12 in field/number.go
github-actions / golangci
[golangci] field/number.go#L12
exported: exported type Int16 should have comment or be unexported (revive)
Raw output
field/number.go:12:6: exported: exported type Int16 should have comment or be unexported (revive)
type Int16 = Number[int16]
^
Check failure on line 14 in field/number.go
github-actions / golangci
[golangci] field/number.go#L14
exported: exported type Int32 should have comment or be unexported (revive)
Raw output
field/number.go:14:6: exported: exported type Int32 should have comment or be unexported (revive)
type Int32 = Number[int32]
^
Check failure on line 16 in field/number.go
github-actions / golangci
[golangci] field/number.go#L16
exported: exported type Int64 should have comment or be unexported (revive)
Raw output
field/number.go:16:6: exported: exported type Int64 should have comment or be unexported (revive)
type Int64 = Number[int64]
^
Check failure on line 18 in field/number.go
github-actions / golangci
[golangci] field/number.go#L18
exported: exported type Uint should have comment or be unexported (revive)
Raw output
field/number.go:18:6: exported: exported type Uint should have comment or be unexported (revive)
type Uint = Number[uint]
^
Check failure on line 20 in field/number.go
github-actions / golangci
[golangci] field/number.go#L20
exported: exported type Uint8 should have comment or be unexported (revive)
Raw output
field/number.go:20:6: exported: exported type Uint8 should have comment or be unexported (revive)
type Uint8 = Number[uint8]
^
Check failure on line 22 in field/number.go
github-actions / golangci
[golangci] field/number.go#L22
exported: exported type Uint16 should have comment or be unexported (revive)
Raw output
field/number.go:22:6: exported: exported type Uint16 should have comment or be unexported (revive)
type Uint16 = Number[uint16]
^
Check failure on line 24 in field/number.go
github-actions / golangci
[golangci] field/number.go#L24
exported: exported type Uint32 should have comment or be unexported (revive)
Raw output
field/number.go:24:6: exported: exported type Uint32 should have comment or be unexported (revive)
type Uint32 = Number[uint32]
^
Check failure on line 26 in field/number.go
github-actions / golangci
[golangci] field/number.go#L26
exported: exported type Uint64 should have comment or be unexported (revive)
Raw output
field/number.go:26:6: exported: exported type Uint64 should have comment or be unexported (revive)
type Uint64 = Number[uint64]
^
Check failure on line 28 in field/number.go
github-actions / golangci
[golangci] field/number.go#L28
exported: exported type Float32 should have comment or be unexported (revive)
Raw output
field/number.go:28:6: exported: exported type Float32 should have comment or be unexported (revive)
type Float32 = Number[float32]
^
Check failure on line 30 in field/number.go
github-actions / golangci
[golangci] field/number.go#L30
exported: exported type Float64 should have comment or be unexported (revive)
Raw output
field/number.go:30:6: exported: exported type Float64 should have comment or be unexported (revive)
type Float64 = Number[float64]
^
Check failure on line 81 in field/time.go
github-actions / golangci
[golangci] field/time.go#L81
exported: exported method Time.Year should have comment or be unexported (revive)
Raw output
field/time.go:81:1: exported: exported method Time.Year should have comment or be unexported (revive)
func (field Time) Year() Number[int] {
^
Check failure on line 40 in field/export.go
github-actions / golangci
[golangci] field/export.go#L40
unexported-return: exported func NewField returns unexported type field.genericsField[database/sql/driver.Valuer], which can be annoying to use (revive)
Raw output
field/export.go:40:53: unexported-return: exported func NewField returns unexported type field.genericsField[database/sql/driver.Valuer], which can be annoying to use (revive)
func NewField(table, column string, opts ...Option) genericsField[driver.Valuer] {
^
Check failure on line 200 in generics.go
github-actions / golangci
[golangci] generics.go#L200
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
Raw output
generics.go:200:9: indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
} else {
return result.(E), nil
}
Check failure on line 209 in generics.go
github-actions / golangci
[golangci] generics.go#L209
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
Raw output
generics.go:209:9: indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
} else {
return result.(E), nil
}
Check failure on line 218 in generics.go
github-actions / golangci
[golangci] generics.go#L218
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
Raw output
generics.go:218:9: indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
} else {
return result.(E), nil
}
Check failure on line 13 in generics.go
github-actions / golangci
[golangci] generics.go#L13
exported: exported type IGenericsDo should have comment or be unexported (revive)
Raw output
generics.go:13:6: exported: exported type IGenericsDo should have comment or be unexported (revive)
type IGenericsDo[T any, E any] interface {
^
Check failure on line 75 in generics.go
github-actions / golangci
[golangci] generics.go#L75
exported: exported type GenericsDo should have comment or be unexported (revive)
Raw output
generics.go:75:6: exported: exported type GenericsDo should have comment or be unexported (revive)
type GenericsDo[T IGenericsDo[T, E], E any] struct {
^