Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-11809

Explicitly enable pep8-naming tests

    Details

    • Templates:
    • Story Points:
      1
    • Team:
      Architecture

      Description

      Currently, flake8 is installed with minimal plugins and does not include pep8-naming. This is problematic on two counts:

      1. The developer guide explicitly specifies naming tests with a set of ignored codes. These codes are not currently checked.
      2. 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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                tjenness Tim Jenness
                Reviewers:
                Jonathan Sick
                Watchers:
                Jonathan Sick, Kian-Tat Lim, Krzysztof Findeisen, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel