Skip to content

Add Naming/FileName rule checking expected class/module per filename

Gabriel Mazetto requested to merge rubocop-naming-filename into master

What does this MR do?

Add a rubocop cop to match filenames with expected Class/Module names to prevent NameError: uninitialized constant because of a failure to require a file using autoload.

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

A CE backport MR will be necessary as well. (https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/17631)

Why was this MR needed?

NameError: uninitialized constant SystemCheck::Geo::FdwSchemaUptoDateCheck when runnning rake gitlab:geo:check was not detected during review process

This can be automated by using Rubocop cops

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/4878

Edited by Gabriel Mazetto

Merge request reports