Page MenuHomeSoftware Heritage

Unify log instruction to use the module logger instance
ClosedPublic

Authored by ardumont on Fri, Oct 1, 12:13 PM.

Details

Summary

This unifies logging instructions as other swh package does it.

This also drops log reference passed to converters.
They were no longer used in the implementation.

Test Plan

tox

Diff Detail

Repository
rDLDG Git loader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D6384 (id=23224)

Rebasing onto 940a5e7e6f...

Current branch diff-target is up to date.
Changes applied before test
commit c7160263202f1a2669931c2e138448e9e1e34438
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Sep 30 20:10:05 2021 +0200

    Unify log instruction to use the module logger instance
    
    This unifies logging instructions as other swh package does it.

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

Update from_disk module as well

Build is green

Patch application report for D6384 (id=23226)

Rebasing onto 940a5e7e6f...

Current branch diff-target is up to date.
Changes applied before test
commit a34aefbf0a7d0647ffcee5fcf99a307bdf1e0298
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Sep 30 20:10:05 2021 +0200

    Unify logging instructions to use module logger instance
    
    This unifies logging instructions with swh packages.

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

This revision is now accepted and ready to land.Fri, Oct 1, 1:29 PM