diff mercurial/phases.py @ 32841: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 511a62669f1b
children e8c8d81eb864
line wrap: on
line diff
--- a/mercurial/phases.py	Fri Apr 10 00:14:16 2015 +0900
+++ b/mercurial/phases.py	Mon Jun 12 16:35:57 2017 -0700
@@ -423,12 +423,12 @@
         if currentphase == oldphase and newphase < oldphase:
             with repo.transaction('pushkey-phase') as tr:
                 advanceboundary(repo, tr, newphase, [bin(nhex)])
-            return 1
+            return True
         elif currentphase == newphase:
             # raced, but got correct result
-            return 1
+            return True
         else:
-            return 0
+            return False
 
 def analyzeremotephases(repo, subset, roots):
     """Compute phases heads and root in a subset of node from root dict