verify: explicitly return 0 if no error are encountered
Relying on the fact None is treated as 0 by other logics seems smarter than we
should be.
--- a/mercurial/verify.py Wed Mar 06 11:35:34 2019 +0100
+++ b/mercurial/verify.py Wed Mar 06 11:38:54 2019 +0100
@@ -134,7 +134,7 @@
This method run all verifications, displaying issues as they are found.
- return 1 if any error have been encountered"""
+ return 1 if any error have been encountered, 0 otherwise."""
repo = self.repo
ui = repo.ui
@@ -171,6 +171,7 @@
ui.warn(_("(first damaged changeset appears to be %d)\n")
% min(self.badrevs))
return 1
+ return 0
def _verifychangelog(self):
ui = self.ui