diff mercurial/obsolete.py @ 32822:e65ff29dbeb0

pushkey: use False/True for return values from push functions It was particularly unclear in phases.pushphase() whether the 0/1 returned were the 0/1 for public/draft phase or for False/True
author Martin von Zweigbergk <martinvonz@google.com>
date Mon, 12 Jun 2017 16:35:57 -0700
parents 5ffb138d7b1a
children 9d472b219fb0
line wrap: on
line diff
--- a/mercurial/obsolete.py	Fri Apr 10 00:14:16 2015 +0900
+++ b/mercurial/obsolete.py	Mon Jun 12 16:35:57 2017 -0700
@@ -944,10 +944,10 @@
     """Push markers over pushkey"""
     if not key.startswith('dump'):
         repo.ui.warn(_('unknown key: %r') % key)
-        return 0
+        return False
     if old:
         repo.ui.warn(_('unexpected old value for %r') % key)
-        return 0
+        return False
     data = util.b85decode(new)
     lock = repo.lock()
     try:
@@ -956,7 +956,7 @@
             repo.obsstore.mergemarkers(tr, data)
             repo.invalidatevolatilesets()
             tr.close()
-            return 1
+            return True
         finally:
             tr.release()
     finally: