Currently, flake8 is installed with minimal plugins and does not include pep8-naming. This is problematic on two counts:
- The developer guide explicitly specifies naming tests with a set of ignored codes. These codes are not currently checked.
- If a developer happens to have pep8-naming installed their build can break because the flake8 testing in CI and in default builds is not testing what it should be.
I am going to add pep8-naming eups package and enable name testing. This will also require fixing any flake8 problems in the packages that have automated flake8 testing.
I do not believe an RFC is required to add this package as the developer guide already implies we are using it.