Patchwork D7152: phabricator: use True primitive instead of b'true' for phabupdate actions

login
register
mail settings
Submitter phabricator
Date Oct. 23, 2019, 9:31 p.m.
Message ID <differential-rev-PHID-DREV-qhaerzudz327hhw4gndz-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/42542/
State Superseded
Headers show

Comments

phabricator - Oct. 23, 2019, 9:31 p.m.
Kwan created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  Something I'd missed in the creatediff port.  This didn't matter before with
  the old PHP form style wireformat, but breaks with the new arcanist format.

REPOSITORY
  rHG Mercurial

BRANCH
  stable

REVISION DETAIL
  https://phab.mercurial-scm.org/D7152

AFFECTED FILES
  hgext/phabricator.py

CHANGE DETAILS




To: Kwan, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/hgext/phabricator.py b/hgext/phabricator.py
--- a/hgext/phabricator.py
+++ b/hgext/phabricator.py
@@ -1613,7 +1613,7 @@ 
 
     actions = []
     for f in flags:
-        actions.append({b'type': f, b'value': b'true'})
+        actions.append({b'type': f, b'value': True})
 
     drevs = querydrev(repo, spec)
     for i, drev in enumerate(drevs):