Patchwork D4047: absorb: note some TODOs from the code review

login
register
mail settings
Submitter phabricator
Date Aug. 6, 2018, 3:54 p.m.
Message ID <7a50f5fee82f4e7232654c084e51c2ef@localhost.localdomain>
Download mbox | patch
Permalink /patch/33339/
State Not Applicable
Headers show

Comments

phabricator - Aug. 6, 2018, 3:54 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGec0697f4f9c9: absorb: note some TODOs from the code review (authored by durin42, committed by ).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D4047?vs=9764&id=9991

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

AFFECTED FILES
  hgext/absorb.py

CHANGE DETAILS




To: durin42, #hg-reviewers, indygreg
Cc: mercurial-devel

Patch

diff --git a/hgext/absorb.py b/hgext/absorb.py
--- a/hgext/absorb.py
+++ b/hgext/absorb.py
@@ -25,6 +25,11 @@ 
     absorb.path = bold
 """
 
+# TODO:
+#  * Rename config items to [commands] namespace
+#  * Converge getdraftstack() with other code in core
+#  * move many attributes on fixupstate to be private
+
 from __future__ import absolute_import
 
 import collections