D4810: repository: introduce an ellipses repo requirement

2018-10-17 Thread martinvonz (Martin von Zweigbergk)
martinvonz added a comment. I think this patch (and later patches that start using it, of course) would be a good idea. I suppose it won't make it into 4.8, but we don't have a freeze, so please send an updated version whenever you have time REPOSITORY rHG Mercurial REVISION DETAIL

D4810: repository: introduce an ellipses repo requirement

2018-10-01 Thread indygreg (Gregory Szorc)
indygreg added a comment. New requirements should be documented in `help/internals/requirements.txt`. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D4810 To: pulkit, #hg-reviewers Cc: indygreg, martinvonz, mercurial-devel

D4810: repository: introduce an ellipses repo requirement

2018-10-01 Thread pulkit (Pulkit Goyal)
pulkit planned changes to this revision. pulkit added a comment. Let's do this step by step and get the wireprotocol command reviewed first. Once that is done, I will resend this one. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D4810 To: pulkit,

D4810: repository: introduce an ellipses repo requirement

2018-09-30 Thread pulkit (Pulkit Goyal)
pulkit added inline comments. INLINE COMMENTS > martinvonz wrote in repository.py:22-24 > I'm not sure I understand what this says. > > The first sentence sounds too obvious to mention (kind of like "when it's > done, it should be working well"), but I suspect you mean something else. Do >

D4810: repository: introduce an ellipses repo requirement

2018-09-30 Thread martinvonz (Martin von Zweigbergk)
martinvonz added inline comments. INLINE COMMENTS > repository.py:22-24 > +# When ellipses and shallowing is finalized and the UI is done, there are no > +# more edge cases where things don't work. This should be generalised to > shallow > +# or ellipses I'm not sure I understand what this

D4810: repository: introduce an ellipses repo requirement

2018-09-30 Thread pulkit (Pulkit Goyal)
pulkit updated this revision to Diff 11501. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D4810?vs=11493=11501 REVISION DETAIL https://phab.mercurial-scm.org/D4810 AFFECTED FILES mercurial/repository.py CHANGE DETAILS diff --git

D4810: repository: introduce an ellipses repo requirement

2018-09-29 Thread pulkit (Pulkit Goyal)
pulkit created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Right now there is no good way to check whether a repo is ellipses enabled or not. This patch introduces a ellipses repo requirement which will be used in