sonar: exclude tests from complexity and parameter rules, raise the S3776 threshold

pti commented

2026-10-19 15:26:41 +02:00

Owner

First SonarQube scan of main reports 209 issues. 144 are go:S3776 cognitive complexity above 15, of which 94 sit in _test.go files, and 5 of the 14 go:S107 too-many-parameters findings are in tests. Table-driven tests are complex by design, so the signal is noise there. Proposal: add sonar.issue.ignore.multicriteria entries in sonar-project.properties scoping go:S3776 and go:S107 to non-test files, and raise the S3776 threshold from 15 to 25 in the Go quality profile on the server, which leaves roughly 37 production functions to refactor. Also set the quality gate to new-code conditions so main stays green while legacy debt is worked down. Part of the SonarQube cleanup tracking issue.

pti referenced this issue from a commit

2026-10-19 15:41:02 +02:00

ci(sonar): scope complexity rules to production code, send the release version

pti referenced this issue

2026-10-19 15:41:14 +02:00

ci/sonar: scope complexity rules to production code, send the release version #75

pti commented

2026-10-19 20:15:18 +02:00

Author

Owner