Simplify metadata within Artifact class

Tom Pollardrequested to merge
tpollard/artifactmetadata into master

Description

Now that !1271 (merged) has landed, we have somewhat simplified that internal handling of artifact metadata by internalising it into the Artifact class from the element, removing the need for accessor methods and less than ideal parameter handling. This simplification can be taken further now that the Artifact instance within element represents a single artifact, with the keys internalised removing the need to have keyed dicts for the metadata. It also allows for the returning of keys to be made redundant in directory methods

Changes proposed in this merge request:

  • metadata not stored in keyed dicts in relation to the artifacts weak & strong key
  • directory methods no longer need to return the used key

This merge request, when approved, will close issue/bug: #1003 (closed)

Merge request reports