Page MenuHomeSoftware Heritage

Index extrinsic metadata from the deposit
ClosedPublic

Authored by vlorentz on Sep 28 2022, 7:02 PM.

Details

Summary

It was added to the metadata_dictionary a while ago, but was excluded
by the indexer itself so far...

Diff Detail

Repository
rDCIDX Metadata indexer
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 31894
Build 49916: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 49915: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D8570 (id=30921)

Could not rebase; Attempt merge onto cdbf090b14...

Updating cdbf090..74867bf
Fast-forward
 swh/indexer/metadata.py                            |   2 +-
 swh/indexer/metadata_dictionary/codemeta.py        |   9 +-
 swh/indexer/metadata_dictionary/github.py          |  13 ++-
 .../tests/metadata_dictionary/test_codemeta.py     |  26 +++++
 .../tests/metadata_dictionary/test_github.py       |  20 +++-
 swh/indexer/tests/test_cli.py                      |   4 +-
 swh/indexer/tests/test_metadata.py                 | 119 +++++++++++++++++++--
 7 files changed, 176 insertions(+), 17 deletions(-)
Changes applied before test
commit 74867bf361709773643d9fa337a166f2d112bf7e
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Wed Sep 28 19:01:52 2022 +0200

    Index extrinsic metadata from the deposit
    
    It was added to the metadata_dictionary a while ago, but was excluded
    by the indexer itself so far...

commit fc6fea5dd87f7a5b73f73e41f94acc9f60d9e96a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Wed Sep 28 18:06:02 2022 +0200

    codemeta: Fix crash when translating PropertyValue objects from codemeta-in-SWORD

commit 6e4c67bede71b43f7b8fe10357065b71616ebc2a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Wed Sep 28 13:40:34 2022 +0200

    github: Ignore archive_url and issues_url; use custom codemeta:issueTracker
    
    The codemeta crosswalk maps archive_url and issues_url to codemeta
    properties, but they are not actually useable because they are
    link templates to API endpoints.

See https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/512/ for more details.

This revision is now accepted and ready to land.Sep 29 2022, 4:49 PM