Skip to content

Do not use require in http handlers (testifylint) (part 2)

Ash McKenzie requested to merge ashmckenzie/fix-linter-errors-3 into master

What does this MR do and why?

⚠ This MR needs Resolve unused-parameter: <specifics> (revive) (!157728 - merged) merged first.

This test-only MR continues on from Do not use require in http handlers (testifylint) (!157724 - merged) by removing a few more go-require: require must only be used in the goroutine running the test function linter violations (am not exactly sure how I missed them in the first place?) 🙂

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Ash McKenzie

Merge request reports