D4200: fastannotate: record some TODO items from indygreg's review
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Thu Aug 9 19:13:34 UTC 2018
durin42 created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Some of these I'll address more or less right away, but some will take
a while for us to resolve.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D4200
AFFECTED FILES
hgext/fastannotate/__init__.py
CHANGE DETAILS
diff --git a/hgext/fastannotate/__init__.py b/hgext/fastannotate/__init__.py
--- a/hgext/fastannotate/__init__.py
+++ b/hgext/fastannotate/__init__.py
@@ -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 _
To: durin42, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list