Page MenuHomeSoftware Heritage

Move Person parsing to swh-model.
ClosedPublic

Authored by vlorentz on Mar 2 2020, 3:59 PM.

Diff Detail

Repository
rDLDG Git loader
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10836
Build 16279: tox-on-jenkinsJenkins
Build 16278: arc lint + arc unit

Event Timeline

olasd added a subscriber: olasd.

(accept pending potential api changes in the base diff)

Do we have that many calls to parse_author that we need to keep the indirection?

If not, then it might make sense to lift the tests out of the way into swh.model, then get rid of the function completely, in a further diff.

This revision is now accepted and ready to land.Mar 3 2020, 1:23 PM
This revision is now accepted and ready to land.Mar 10 2020, 10:45 AM
This revision was automatically updated to reflect the committed changes.