D4903: narrow: update TODO.rst now that we filter status in ctx

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Fri Oct 5 23:07:22 UTC 2018


martinvonz created this revision.
Herald added a reviewer: durin42.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  The comment referred to was addressed in https://phab.mercurial-scm.org/rHGe411774a2e0f0a616f44bab787cbbf80ed5f7f48 (narrow: move
  status-filtering to core and to ctx, 2018-08-02). I also think
  https://phab.mercurial-scm.org/rHG84092edd5c88bbb3c2a5de377cfd9ca79019fa39 (narrow: drop unnecessary overrides of patch, 2018-09-28)
  suggests that it was the right thing to do.

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  hgext/narrow/TODO.rst

CHANGE DETAILS

diff --git a/hgext/narrow/TODO.rst b/hgext/narrow/TODO.rst
--- a/hgext/narrow/TODO.rst
+++ b/hgext/narrow/TODO.rst
@@ -1,11 +1,6 @@
 Integration with the share extension needs improvement. Right now
 we've seen some odd bugs.
 
-Resolve commentary on narrowrepo.wraprepo.narrowrepository.status
-about the filtering of status being done at an awkward layer. This
-came up the import to hgext, but nobody's got concrete improvement
-ideas as of then.
-
 Address commentary in manifest.excludedmanifestrevlog.add -
 specifically we should improve the collaboration with core so that
 add() never gets called on an excluded directory and we can improve



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


More information about the Mercurial-devel mailing list