Skip to content

dev/sg: remove lint.Report.Duration, clarify usage of Output and Err

Administrator requested to merge sg-lint-report into main

Created by: bobheadxi

Setting Report.Duration is a lot of boilerplate in non-script lint.Runner for something that could easily be unified, so this changes sg lint to automatically infer a duration.

The first few times I built linters when Output and Err gets used was pretty confusing - IMO there's some room for improvement here still, but I just added some docs and a helper to get the summary to try and clarify a bit.

Test plan

go run ./dev/sg lint

Merge request reports