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

Make afw PEP8 compliant and enable auto testing

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw

      Description

      Fix PEP8 warnings and enable automatic testing by scons and by Travis on github.

      I also fixed the Doxygen warnings, as the warnings about duplicate section names were showing up when building other packages, which was very annoying. One change was to stop Doxygen from scanning any C++ source files in src/.... I moved the Doxygen documentation from files to the headers long ago, and scanning the source was a frequent cause of spurious Doxygen warnings.

        Attachments

          Issue Links

          is triggering

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-471 Change the Python style guide to match PEP8 handling of "="

          • Implemented
          relates to

          Improvement - An improvement or enhancement to an existing feature or task. DM-14037 Use EXCLUDE_PATTERNS more safely afw Doxygen config

          • Done

            Activity

            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Some excellent changes here. Thank you. Kian-Tat Lim these changes do assume that RFC-471 will pass so I imagine you would like to hold off merging? The trouble with afw is the moving target so it's hard to merge the longer we delay.

            Show
            tjenness Tim Jenness added a comment - Some excellent changes here. Thank you. Kian-Tat Lim these changes do assume that RFC-471 will pass so I imagine you would like to hold off merging? The trouble with afw is the moving target so it's hard to merge the longer we delay.

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Tim Jenness
              Watchers:
              Kian-Tat Lim, Russell Owen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.