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

          Container Issues

            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