Page MenuHomeSoftware Heritage

deposit: Add utils to parse metadata provenance from atom entry file
ClosedPublic

Authored by ardumont on Feb 18 2022, 2:38 PM.

Details

Diff Detail

Repository
rDDEP Push deposit
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26984
Build 42191: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 42190: arc lint + arc unit

Event Timeline

Build has FAILED

Patch application report for D7200 (id=26110)

Rebasing onto c30dc3da46...

Current branch diff-target is up to date.
Changes applied before test
commit 8def64d06183a0a42b4102e9e262866d9d4789f0
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Feb 18 14:35:49 2022 +0100

    deposit: Add utils to parse metadata provenance from atom entry file
    
    Related to T3677

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

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 18 2022, 2:41 PM
Harbormaster failed remote builds in B26983: Diff 26110!

Fix docs which failed the build

Build is green

Patch application report for D7200 (id=26111)

Rebasing onto c30dc3da46...

Current branch diff-target is up to date.
Changes applied before test
commit 2857213c51cc11240b090b3a85b41efa4acc6a66
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Feb 18 14:35:49 2022 +0100

    deposit: Add utils to parse metadata provenance from atom entry file
    
    Related to T3677

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

swh/deposit/utils.py
181

"liar liar, pants on fire"

swh/deposit/utils.py
181

haha

hmm I'll probably rewrite all these utils with XPath at some point

This revision is now accepted and ready to land.Feb 21 2022, 1:41 PM

hmm I'll probably rewrite all these utils with XPath at some point

yeah, that'd be fine.

Do you have a specific library in mind?

Build is green

Patch application report for D7200 (id=26128)

Rebasing onto e1549dc0d3...

Current branch diff-target is up to date.
Changes applied before test
commit 40adc8c23b98b24bfdc5526d9e6b8bfe2d198c39
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Feb 18 14:35:49 2022 +0100

    deposit: Add utils to parse metadata provenance from atom entry file
    
    Related to T3677

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