Skip to content

Commit d801080

Browse files
committed
[contrib] fix lints
1 parent a0feee9 commit d801080

2 files changed

Lines changed: 8 additions & 5 deletions

File tree

contrib/.golangci.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ linters:
2828
- ifshort
2929
- deadcode
3030
- structcheck
31-
fast: true
31+
- gocritic
32+
- exhaustive
33+
fast: false
3234
linters-settings:
3335
# decorder:
3436
# # Required order of `type`, `const`, `var` and `func` declarations inside a file.
@@ -56,5 +58,6 @@ linters-settings:
5658
# Such cases aren't reported by default.
5759
# Default: false
5860
check-blank: true
59-
61+
cyclop:
62+
max-complexity: 15
6063

contrib/swagger/swagger.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -224,11 +224,13 @@ func toSwagDefinition(m desc.ParsedMessage) spec.Schema {
224224
}
225225

226226
for fields.Back() != nil {
227-
p := fields.Remove(fields.Back()).(desc.ParsedField) //nolint:forcetypeassert
227+
p := fields.Remove(fields.Back()).(desc.ParsedField) //nolint:errcheck,forcetypeassert
228228

229229
name, kind, wrapFuncChain := getWrapFunc(p)
230230

231231
switch kind {
232+
default:
233+
def.SetProperty(p.Name, wrapFuncChain.Apply(spec.StringProperty()))
232234
case desc.Object:
233235
if p.Embedded {
234236
for _, f := range p.Message.Fields {
@@ -247,8 +249,6 @@ func toSwagDefinition(m desc.ParsedMessage) spec.Schema {
247249
def.SetProperty(p.Name, wrapFuncChain.Apply(spec.BoolProperty()))
248250
case desc.Byte:
249251
def.SetProperty(p.Name, wrapFuncChain.Apply(spec.Int8Property()))
250-
default:
251-
def.SetProperty(p.Name, wrapFuncChain.Apply(spec.StringProperty()))
252252
}
253253
}
254254

0 commit comments

Comments
 (0)