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