changeset 39212:ca053fc7efc5

fastannotate: record some TODO items from indygreg's review Some of these I'll address more or less right away, but some will take a while for us to resolve. Differential Revision: https://phab.mercurial-scm.org/D4200
author Augie Fackler <augie@google.com>
date Thu, 09 Aug 2018 14:56:29 -0400
parents c8a40b33ce30
children 303dae0136b0
files hgext/fastannotate/__init__.py
diffstat 1 files changed, 26 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/hgext/fastannotate/__init__.py	Wed Aug 01 11:10:41 2018 -0400
+++ b/hgext/fastannotate/__init__.py	Thu Aug 09 14:56:29 2018 -0400
@@ -98,6 +98,32 @@
     perfhack = True
 """
 
+# TODO from import:
+# * `branch` is probably the wrong term, throughout the code.
+#
+# * replace the fastannotate `modes` configuration with a collection
+#   of booleans.
+#
+# * Use the templater instead of bespoke formatting
+#
+# * rename the config knob for updating the local cache from a remote server
+#
+# * remove the remotefilelog-peer-sharing functionality
+#
+# * move various global-setup bits to extsetup() or reposetup()
+#
+# * assume repo.requirements will always exist
+#
+# * move `flock` based locking to a common area
+#
+# * revise wireprotocol for sharing annotate files
+#
+# * figure out a sensible default for `mainbranch` (with the caveat
+#   that we probably also want to figure out a better term than
+#   `branch`, see above)
+#
+# * format changes to the revmap file (maybe use length-encoding
+#   instead of null-terminated file paths at least?)
 from __future__ import absolute_import
 
 from mercurial.i18n import _