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

Linter errors in daf_persistence

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_persistence
    • Labels:

      Description

      Running flake8 on daf_persistence as part of DM-7069 I find two real errors.

      1. in butler.py the key argument was removed from queryMetadata in commit 3a169f but it is still documented as being there and is still used inside the function.
      2. mapper.py refers to class RepositoryMapperCfg but that class is not seemingly defined anywhere.

        Attachments

          Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. DM-9774 Undefined keyword in Butler method

          • Invalid

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7894 mapper and butler queryMetadata method badly documented

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9255 Butler.queryMetadata doc/API/implementation mismatch

          • Invalid

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            bvan Brian Van Klaveren added a comment -

            Looks fine.

            Show
            bvan Brian Van Klaveren added a comment - Looks fine.
            Hide
            Permalink
            npease Nate Pease [X] (Inactive) added a comment -

            Brian Van Klaveren says looks fine.

            Show
            npease Nate Pease [X] (Inactive) added a comment - Brian Van Klaveren says looks fine.

              People

              Assignee:
              npease Nate Pease [X] (Inactive)
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Brian Van Klaveren
              Watchers:
              Brian Van Klaveren, Nate Pease [X] (Inactive), Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.