Page MenuHomeSoftware Heritage

replay: Improve signature of DirEditor.open_file method
ClosedPublic

Authored by anlambert on Feb 4 2022, 2:29 PM.

Details

Summary

Add path parameter for consistency with other methods.

Depends on D7086

Diff Detail

Repository
rDLDSVN Subversion (SVN) 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 D7087 (id=25705)

Could not rebase; Attempt merge onto 09e83fd94c...

Updating 09e83fd..651bb4e
Fast-forward
 requirements-test.txt    |   2 +-
 swh/loader/svn/loader.py |   6 --
 swh/loader/svn/replay.py | 210 ++++++++++++++++++++++++-----------------------
 swh/loader/svn/svn.py    |   9 +-
 4 files changed, 117 insertions(+), 110 deletions(-)
Changes applied before test
commit 651bb4e0b5c42ac67a8c29b2da9986261e9ff682
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Feb 4 14:27:36 2022 +0100

    replay: Improve signature of DirEditor.open_file method
    
    Add path parameter for consistency with other methods.

commit d5bdea55bc527a94d9e735648a2b835ea2510fba
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Feb 4 14:07:57 2022 +0100

    replay: Move external processing in a dedicated method
    
    That code grew a lot so better putting it in a dedicated method, plus
    it enables to remove one indentation level for better readability.

commit 369fbea00ea54b202a08db8541a7e6db42a0cc95
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Feb 4 13:48:00 2022 +0100

    svn: Add debug log before executing checkout or export operation

commit 8bde0663cb4a72a80212e7bab21a962ddf510e25
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Fri Feb 4 13:40:52 2022 +0100

    requirements-test: Pin pytest to < 7.0.0
    
    pytest v7.0.0 removed _pytest.tmpdir.TempdirFactory, which is used
    by some of the pytest plugins we use.

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

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