Skip to content

Enable shadowing checking in govet

Pedro Pombeiro (OOO from July 16th till Aug 7th) requested to merge 25906-govet-enable-shadowing-check into master

What does this MR do?

Enables the check-shadowing check of the govet linter.

Why was this MR needed?

There have been bugs/broken tests caused by inadvertent shadowing of variables. Enabling this linter will make us more aware of a choice we need to make.

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Closes #25906 (closed)

Edited by Georgi N. Georgiev

Merge request reports