forked from vikunja/vikunja
ad04d302af
This way the config module can already use the log module with the same result (default logging to StdOut with Level INFO, same output as before) but ENV variables can already change the logging of config file related log output). It is now possible to dump as a cronjob without having to filter the default log about the used config file. Also: - all logging modules are now configurable when initializing which makes testing easier - viper dependency removed from logging - log correct settings when configured error level is invalid - deprecation of value "false" for log.standard and log.events (already not mentioned in https://vikunja.io/docs/config-options/) Co-authored-by: Berengar W. Lehr <Berengar.Lehr@uni-jena.de> Reviewed-on: vikunja/api#1606 Reviewed-by: konrad <k@knt.li> Co-authored-by: Peter H0ffmann <hoffmannp@noreply.kolaente.de> Co-committed-by: Peter H0ffmann <hoffmannp@noreply.kolaente.de>
109 lines
2.5 KiB
YAML
109 lines
2.5 KiB
YAML
run:
|
|
timeout: 15m
|
|
tests: true
|
|
|
|
linters:
|
|
enable:
|
|
- megacheck
|
|
- govet
|
|
- goconst
|
|
- gocritic
|
|
- gocyclo
|
|
- goerr113
|
|
- goheader
|
|
- gofmt
|
|
- goimports
|
|
- revive
|
|
- misspell
|
|
disable:
|
|
- scopelint # Obsolete, using exportloopref instead
|
|
- durationcheck
|
|
presets:
|
|
- bugs
|
|
- unused
|
|
fast: false
|
|
|
|
linters-settings:
|
|
nestif:
|
|
min-complexity: 6
|
|
goheader:
|
|
template-path: code-header-template.txt
|
|
|
|
issues:
|
|
exclude-rules:
|
|
# Exclude some linters from running on tests files.
|
|
- path: _test\.go
|
|
linters:
|
|
- gocyclo
|
|
- deadcode
|
|
- errorlint
|
|
- path: pkg/integrations/*
|
|
linters:
|
|
- gocyclo
|
|
- deadcode
|
|
- varcheck
|
|
- unparam
|
|
- bodyclose
|
|
- path: pkg/integrations/*
|
|
text: "unlambda"
|
|
linters:
|
|
- gocritic
|
|
- path: pkg/modules/background/unsplash/unsplash\.go
|
|
linters:
|
|
- bodyclose
|
|
- path: pkg/migration/*
|
|
linters:
|
|
- exhaustive
|
|
- goconst
|
|
- goerr113
|
|
- path: pkg/models/task_collection_filter\.go
|
|
linters:
|
|
- exhaustive
|
|
- path: pkg/utils/random_string\.go
|
|
text: "G404:" # We don't care about cryptographically secure randomness when we're using that utility function.
|
|
linters:
|
|
- gosec
|
|
- path: pkg/modules/dump/*
|
|
linters:
|
|
- goerr113
|
|
- path: pkg/
|
|
text: "err113: do not define dynamic errors, use wrapped static errors instead:"
|
|
linters:
|
|
- goerr113
|
|
- text: "commentFormatting: put a space between `//` and comment text"
|
|
linters:
|
|
- gocritic
|
|
- path: pkg/modules/migration
|
|
linters:
|
|
- gocyclo
|
|
- path: pkg/routes/api/v1/docs.go
|
|
linters:
|
|
- goheader
|
|
- misspell
|
|
- gosmopolitan
|
|
- text: "Missed string"
|
|
linters:
|
|
- goheader
|
|
- path: pkg/.*/error.go
|
|
linters:
|
|
- errorlint
|
|
- path: pkg/models/favorites\.go
|
|
linters:
|
|
- nilerr
|
|
- path: pkg/models/project\.go
|
|
text: "string `parent_project_id` has 3 occurrences, make it a constant"
|
|
- path: pkg/models/events\.go
|
|
linters:
|
|
- musttag
|
|
- path: pkg/models/task_collection.go
|
|
text: 'append result not assigned to the same slice'
|
|
- text: 'string `label_id` has 3 occurrences, make it a constant'
|
|
linters:
|
|
- goconst
|
|
- text: 'string `labels` has 3 occurrences, make it a constant'
|
|
linters:
|
|
- goconst
|
|
- text: 'string `off` has 6 occurrences, make it a constant'
|
|
linters:
|
|
- goconst
|