Page MenuHomeSoftware Heritage

mailmaps: Add an endpoint to fetch the list
ClosedPublic

Authored by vlorentz on Feb 7 2022, 2:50 PM.

Details

Summary

This will be useful for debugging and checking records are indeed
configured.

Diff Detail

Repository
rDWAPPS Web applications
Branch
mailmap
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26576
Build 41567: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 41566: arc lint + arc unit

Event Timeline

Build has FAILED

Patch application report for D7095 (id=25747)

Rebasing onto d4ca7bede2...

Current branch diff-target is up to date.
Changes applied before test
commit 98bdf12aea75bdcc2e3e96b49515112987daa164
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Feb 7 14:49:54 2022 +0100

    mailmaps: Add an endpoint to fetch the list
    
    This will be useful for debugging and checking records are indeed
    configured.

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

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 7 2022, 3:10 PM
Harbormaster failed remote builds in B26576: Diff 25747!

Build is green

Patch application report for D7095 (id=25747)

Rebasing onto d4ca7bede2...

Current branch diff-target is up to date.
Changes applied before test
commit 98bdf12aea75bdcc2e3e96b49515112987daa164
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Feb 7 14:49:54 2022 +0100

    mailmaps: Add an endpoint to fetch the list
    
    This will be useful for debugging and checking records are indeed
    configured.

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

anlambert added a subscriber: anlambert.

Looks good to me.

swh/web/tests/auth/test_mailmap.py
52–53

Ah right it crashes because the endpoint does not have a HTML doc declared with apidoc decorator
and check_api_get_responses checks the text/html content type..

You can remove the TODO I think.

This revision is now accepted and ready to land.Feb 7 2022, 6:07 PM
This revision was landed with ongoing or failed builds.Feb 7 2022, 7:10 PM
This revision was automatically updated to reflect the committed changes.