Page MenuHomeSoftware Heritage

sword mapping: Add support for legacy JSON-encoded Atom documents
ClosedPublic

Authored by vlorentz on Aug 12 2022, 11:48 AM.

Details

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

Build has FAILED

Patch application report for D8238 (id=29716)

Could not rebase; Attempt merge onto bb9082a6e5...

Updating bb9082a..e39f5e5
Fast-forward
 docs/metadata-workflow.rst                         |   2 +-
 swh/indexer/codemeta.py                            |   5 +-
 swh/indexer/metadata_dictionary/__init__.py        |   2 +
 swh/indexer/metadata_dictionary/codemeta.py        | 126 +++++++++++++-
 .../tests/metadata_dictionary/test_codemeta.py     | 192 +++++++++++++++++++++
 swh/indexer/tests/test_cli.py                      |  15 +-
 6 files changed, 335 insertions(+), 7 deletions(-)
Changes applied before test
commit e39f5e52c0bee936c89147637eeb277fdd3b71a6
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:48:17 2022 +0200

    sword mapping: Add support for legacy JSON-encoded Atom documents

commit 03d8e1cbc5f863133c837cfb0ff394b7a658ce4d
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:47:26 2022 +0200

    sword mapping: Add support for Atom author elements
    
    Atom documents from old deposits usually use these instead of codemeta's

commit 30d195ccd64ba1c79379d444dff108a2314bc3b0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 11 11:57:52 2022 +0200

    metadata_dictionary: Add mapping for SWORD/Atom with Codemeta

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/417/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/417/console

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 12 2022, 11:57 AM
Harbormaster failed remote builds in B30768: Diff 29716!

Build has FAILED

Patch application report for D8238 (id=29717)

Could not rebase; Attempt merge onto bb9082a6e5...

Updating bb9082a..cca14fb
Fast-forward
 docs/metadata-workflow.rst                         |   2 +-
 swh/indexer/codemeta.py                            |   5 +-
 swh/indexer/metadata_dictionary/__init__.py        |   2 +
 swh/indexer/metadata_dictionary/codemeta.py        | 126 +++++++++++++-
 .../tests/metadata_dictionary/test_codemeta.py     | 192 +++++++++++++++++++++
 swh/indexer/tests/test_cli.py                      |  15 +-
 6 files changed, 335 insertions(+), 7 deletions(-)
Changes applied before test
commit cca14fba9bb7ecb0e1542e943b48717b63580217
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:48:17 2022 +0200

    sword mapping: Add support for legacy JSON-encoded Atom documents

commit 03d8e1cbc5f863133c837cfb0ff394b7a658ce4d
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:47:26 2022 +0200

    sword mapping: Add support for Atom author elements
    
    Atom documents from old deposits usually use these instead of codemeta's

commit 30d195ccd64ba1c79379d444dff108a2314bc3b0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 11 11:57:52 2022 +0200

    metadata_dictionary: Add mapping for SWORD/Atom with Codemeta

Link to build: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/418/
See console output for more information: https://jenkins.softwareheritage.org/job/DCIDX/job/tests-on-diff/418/console

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 12 2022, 12:18 PM
Harbormaster failed remote builds in B30769: Diff 29717!

I can't reproduce this build failure :/

it seems unrelated, anyway

Build is green

Patch application report for D8238 (id=29936)

Could not rebase; Attempt merge onto e8cafad61c...

Updating e8cafad..a0fdaed
Fast-forward
 docs/metadata-workflow.rst                         |   2 +-
 swh/indexer/codemeta.py                            |   5 +-
 swh/indexer/metadata_dictionary/__init__.py        |   2 +
 swh/indexer/metadata_dictionary/codemeta.py        | 126 +++++++++++++-
 .../tests/metadata_dictionary/test_codemeta.py     | 192 +++++++++++++++++++++
 swh/indexer/tests/test_cli.py                      |  15 +-
 6 files changed, 335 insertions(+), 7 deletions(-)
Changes applied before test
commit a0fdaedda5636042735d6536da23e4a0786545a4
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:48:17 2022 +0200

    sword mapping: Add support for legacy JSON-encoded Atom documents

commit 13e9bf18c2d6c138183ef42af6928cea473b75b0
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 12 11:47:26 2022 +0200

    sword mapping: Add support for Atom author elements
    
    Atom documents from old deposits usually use these instead of codemeta's

commit 90dad275308fd0d7e70cb3a50cd395912f0da733
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 11 11:57:52 2022 +0200

    metadata_dictionary: Add mapping for SWORD/Atom with Codemeta

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

This revision was not accepted when it landed; it landed in state Needs Review.Aug 25 2022, 11:57 AM
This revision was automatically updated to reflect the committed changes.