It was added to the metadata_dictionary a while ago, but was excluded
by the indexer itself so far...
Details
Details
- Reviewers
ardumont - Group Reviewers
Reviewers - Maniphest Tasks
- T4401: Index metadata from the deposit
- Commits
- rDCIDX74867bf36170: Index extrinsic metadata from the deposit
Diff Detail
Diff Detail
- Repository
- rDCIDX Metadata indexer
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
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.