golangci-lint - хочу "действительно" игнорировать файл go, а не просто анализировать его и подавлять предупреждения - мой текущий подход игнорирования пожирает память и работает медленно - PullRequest
1 голос
/ 22 января 2020

С golangci-lint Я знаю, что могу использовать настройки skip-dirs или skip-files, но они просто перестают сообщать в конце , и я думаю, что инструмент все еще "работает", цитируя документы :

... какие файлы пропустить: они будут проанализированы, но о проблемах с ними не будет сообщено. ...

Есть ли, скажем, вариант //nolint, чтобы в первую очередь предотвратить анализ некоторых файлов с помощью golangci-lint, чтобы мы не тратили ресурсы времени сборки - память / CPU - на них их?

Я также пробовал строки вверху, например // Code generated by XXX. DO NOT EDIT., и такие, как //nolint, прекращают отчеты, но я думаю, что golangci-lint все еще выполняет анализ - т.е. инструмент работает медленно с большим объемом памяти, когда большие «игнорируемые» файлы присутствуют. С https://github.com/golangci/golangci-lint#nolint я могу сделать

> //nolint:unparam    
> package pkg

И действительно, об ошибках ворса не сообщается, НО я все еще получаю чрезмерное использование памяти, например

13:14 $ golangci-lint run --timeout 30m -v 
INFO [config_reader] Config search paths: [ REMOVED FOR STACKOVERFLOW POST] 
INFO [config_reader] Used config file .golangci.yml 
INFO [lintersdb] Active 17 linters: [deadcode dogsled errcheck gocyclo golint gosimple govet ineffassign misspell nakedret scopelint staticcheck structcheck typecheck unconvert unused varcheck] 
INFO [lintersdb] Active 17 linters: [deadcode dogsled errcheck gocyclo golint gosimple govet ineffassign misspell nakedret scopelint staticcheck structcheck typecheck unconvert unused varcheck] 
INFO [loader] Go packages loading at mode 575 (deps|imports|name|types_sizes|compiled_files|exports_file|files) took 15.003932542s 
INFO [runner/filename_unadjuster] Pre-built 0 adjustments in 831.549179ms 
INFO [runner/unused/goanalysis] analyzers took 15.395924718s with top 10 stages: U1000: 13.12191427s, buildssa: 2.274010448s 
INFO [runner/goanalysis_metalinter/goanalysis] analyzers took 8m25.344883444s with top 10 stages: golint: 6m29.017594865s, ineffassign: 50.003663795s, vrp: 34.694968113s, misspell: 5.91752885s, buildssa: 4.09136586s, SA2003: 1.003697913s, copylocks: 660.780161ms, SA4000: 601.476823ms, SA4018: 510.871677ms, SA4004: 476.466048ms 
INFO [runner] Issues before processing: 28876, after processing: 0 
INFO [runner] Processors filtering stat (out/in): skip_files: 28876/28876, skip_dirs: 28876/28876, identifier_marker: 53/53, path_prettifier: 28876/28876, autogenerated_exclude: 53/28876, exclude: 0/53, filename_unadjuster: 28876/28876, cgo: 28876/28876 
INFO [runner] processing took 60.293318ms with stages: autogenerated_exclude: 33.592255ms, path_prettifier: 18.17708ms, skip_dirs: 4.421717ms, filename_unadjuster: 1.497968ms, cgo: 1.333012ms, identifier_marker: 1.036647ms, exclude: 220.939µs, nolint: 8.996µs, max_same_issues: 1.448µs, uniq_by_line: 592ns, exclude-rules: 544ns, max_from_linter: 467ns, skip_files: 386ns, diff: 362ns, path_shortener: 331ns, source_code: 309ns, max_per_file_from_linter: 265ns 
INFO [runner] linters took 6m47.422489108s with stages: goanalysis_metalinter: 6m28.298523575s, unused: 19.033707326s 
INFO File cache stats: 2 entries of total size 40.4MiB 
INFO Memory: 4114 samples, avg is 7797.4MB, max is 8629.2MB 
INFO Execution took 7m3.337365705s

Когда я удаляю большие файлы golangci-lint ведет себя нормально.

1 Ответ

1 голос
/ 22 января 2020

Использование нового тега сборки для исключения файлов для golangci-lint на самом деле уменьшает потребление памяти (предотвращая синтаксический анализ файлов)

Пример ниже основан на: https://android.googlesource.com/platform/external/syzkaller/+/01d1494471798d6b91af9813d1d6c3e13a55536c

Go файлов, которые теперь будут «эффективно» пропущены

 // AUTOGENERATED FILE

 // +build !codeanalysis

 package foo

.golangci.yml

run:
  deadline: 8m
  skip-dirs:
    - pkg/kd
  # Autogenerated files take too much time and memory to load,
  # even if we skip them with skip-dirs.
  # So we define this tag and use it in the autogenerated files.
  build-tags:
    - codeanalysis

Я кратко проверил это с моей настройкой, и инструмент больше не потребляет много памяти, в отличие от следующих подходов ...

  • //nolint
  • skip-dirs или настройки skip-файлов
  • // Code generated by XXX. DO NOT EDIT. строк для отображения файлов генерируются
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...