diff --git a/.golangci.yml b/.golangci.yml index ed78d9d..281dc77 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -91,40 +91,12 @@ linters: - govet text: "unusedwrite" path: _test\.go - - linters: - - errcheck - - gosec - path: internal/testutil/ - - linters: - - all - path: .*_gen\.go - # revive var-naming: "api" is an intentional package name for the HTTP API - # layer; it is not a utility package and "avoid meaningless package names" - # is a false positive here. All 88+ files in internal/api use this name. - - linters: - - revive - path: internal/api/ - text: "avoid meaningless package names" - # noctx: exec.Command calls in configure CLI helpers intentionally - # omit context (CommandRunner interface contract; operator-controlled inputs). - # Suppressed per-file rather than per-line so #nosec G204 can lead the comment. - - linters: - - noctx - path: (cmd/configure_gcp|cmd/configure_azure)\.go # noctx: http.Get and net.Listen in test helpers appropriately use the # background context; threading a test context through every helper would # add noise without improving test correctness. - linters: - noctx path: _test\.go - # misspell: "cancelled" is the canonical DB column name (migration 000035, - # UK spelling throughout; migration #1277 tracks the rename). "initialised" - # appears in established comments in internal/config/types.go matching the - # existing UK convention. Allow both spellings in the files listed below. - - linters: - - misspell - path: (internal/config/(store_postgres|store_postgres_pgxmock_test|store_postgres_ladder|types|interfaces)|pkg/ladder/(store|types_test)|internal/api/(handler_history|handler_ri_exchange(_test)?|router_handlers_test|coverage_extras_test)|internal/mocks/stores|internal/purchase/scheduled_fire_test)\.go - text: "cancelled|initialised" paths: - third_party$ - builtin$