The update_checksum method does nothing so better removing it.
Details
Details
- Reviewers
ardumont - Group Reviewers
Reviewers - Commits
- rDLDSVNd614911abfba: ra: Remove no-op method in DirEditor
Diff Detail
Diff Detail
- Repository
- rDLDSVN Subversion (SVN) loader
- Branch
- ra-remove-update-checksum
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 25819 Build 40354: Phabricator diff pipeline on jenkins Jenkins console · Jenkins Build 40353: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Patch application report for D6881 (id=24948)
Rebasing onto 196c9d596b...
Current branch diff-target is up to date.
Changes applied before test
commit d614911abfbab5c1c6e4a0d704dcc4fb000be2e9
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date: Wed Jan 5 14:51:38 2022 +0100
ra: Remove no-op method in DirEditor
The update_checksum method does nothing so better removing it.See https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/219/ for more details.
Comment Actions
Thanks.
I guess it has something to do with your class merge last month or so (on that module).
That's no longer useful so yeah, let's drop it.
Comment Actions
Yes, I think it is a remain from the past but indeed I forgot to remove it when I merged the classes.