headscale/.golangci.yaml

---
run:
  timeout: 10m
  build-tags:
    - ts2019

issues:
  skip-dirs:
    - gen
linters:
  enable-all: true
  disable:
    - depguard

    - exhaustivestruct
    - revive
    - lll
    - interfacer
    - scopelint
    - maligned
    - golint
    - gofmt
    - gochecknoglobals
    - gochecknoinits
    - gocognit
    - funlen
    - exhaustivestruct
    - tagliatelle
    - godox
    - ireturn
    - execinquery
    - exhaustruct
    - nolintlint
    - musttag # causes issues with imported libs
    - depguard

    # deprecated
    - structcheck # replaced by unused
    - ifshort # deprecated by the owner
    - varcheck # replaced by unused
    - nosnakecase # replaced by revive
    - deadcode # replaced by unused

    # We should strive to enable these:
    - wrapcheck
    - dupl
    - makezero
    - maintidx

    # Limits the methods of an interface to 10. We have more in integration tests
    - interfacebloat

    # We might want to enable this, but it might be a lot of work
    - cyclop
    - nestif
    - wsl # might be incompatible with gofumpt
    - testpackage
    - paralleltest

linters-settings:
  varnamelen:
    ignore-type-assert-ok: true
    ignore-map-index-ok: true
    ignore-names:
      - err
      - db
      - id
      - ip
      - ok
      - c
      - tt

  gocritic:
    disabled-checks:
      - appendAssign
      # TODO(kradalby): Remove this
      - ifElseChain
Add golangcilint config 2021-10-27 03:07:19 -04:00			`---`
			`run:`
Align with update golangci-lint 2021-11-13 04:11:03 -05:00			`timeout: 10m`
Added ts2019 buildtag to CI config Otherwise we are getting utils.go:119:6: `decode` is unused (deadcode) Signed-off-by: Juan Font Alonso <juanfontalonso@gmail.com> 2022-11-13 07:34:08 -05:00			`build-tags:`
			`- ts2019`
Add gen explicitly to skip list 2021-10-27 03:08:24 -04:00
			`issues:`
			`skip-dirs:`
			`- gen`
Setup more linters and goals for golangci 2021-11-13 03:53:34 -05:00			`linters:`
			`enable-all: true`
			`disable:`
add annoying linter to golangci Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com> 2023-07-17 05:12:59 -04:00			`- depguard`

Setup more linters and goals for golangci 2021-11-13 03:53:34 -05:00			`- exhaustivestruct`
			`- revive`
			`- lll`
			`- interfacer`
			`- scopelint`
			`- maligned`
			`- golint`
			`- gofmt`
			`- gochecknoglobals`
			`- gochecknoinits`
			`- gocognit`
			`- funlen`
			`- exhaustivestruct`
			`- tagliatelle`
			`- godox`
fix: revert previous commit and add exclusion of linter 2022-05-16 08:29:05 -04:00			`- ireturn`
			`- execinquery`
fix: ignore exhaust linter 2022-05-16 09:13:16 -04:00			`- exhaustruct`
fix bug in #912 (#914) 2022-11-05 04:07:22 -04:00			`- nolintlint`
Remove deprecated linters + one casuing issues with imports 2023-04-04 16:24:47 -04:00			`- musttag # causes issues with imported libs`
General fixups discovered by checking errors There was a lot of tests that actually threw a lot of errors and that did not pass all the way because we didnt check everything. This commit should fix all of these cases. Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com> 2023-08-31 12:37:18 -04:00			`- depguard`
Remove deprecated linters + one casuing issues with imports 2023-04-04 16:24:47 -04:00
			`# deprecated`
			`- structcheck # replaced by unused`
			`- ifshort # deprecated by the owner`
			`- varcheck # replaced by unused`
			`- nosnakecase # replaced by revive`
			`- deadcode # replaced by unused`
Setup more linters and goals for golangci 2021-11-13 03:53:34 -05:00
			`# We should strive to enable these:`
			`- wrapcheck`
			`- dupl`
Remove unused params or returns 2021-11-14 12:03:21 -05:00			`- makezero`
fix: revert previous commit and add exclusion of linter 2022-05-16 08:29:05 -04:00			`- maintidx`
Setup more linters and goals for golangci 2021-11-13 03:53:34 -05:00
disable interfacebloat linter 2022-11-13 11:02:42 -05:00			`# Limits the methods of an interface to 10. We have more in integration tests`
			`- interfacebloat`

Setup more linters and goals for golangci 2021-11-13 03:53:34 -05:00			`# We might want to enable this, but it might be a lot of work`
			`- cyclop`
			`- nestif`
Move wsl, might not be feasible 2021-11-14 10:49:54 -05:00			`- wsl # might be incompatible with gofumpt`
Add and fix errorlint 2021-11-15 11:26:41 -05:00			`- testpackage`
Add and fix forcetypeassert 2021-11-15 13:42:44 -05:00			`- paralleltest`
Add exceptions to var name length 2021-11-15 11:16:16 -05:00
			`linters-settings:`
			`varnamelen:`
			`ignore-type-assert-ok: true`
			`ignore-map-index-ok: true`
			`ignore-names:`
			`- err`
			`- db`
			`- id`
			`- ip`
			`- ok`
			`- c`
chore(linter): ignore tt var as it's generated code (vscode) 2022-02-21 10:10:20 -05:00			`- tt`
Fix gocritic 2021-11-24 05:13:41 -05:00
			`gocritic:`
			`disabled-checks:`
			`- appendAssign`
			`# TODO(kradalby): Remove this`
			`- ifElseChain`