martinvonz added inline comments.

INLINE COMMENTS

> pulkit wrote in narrowcommands.py:331
> Following pattern of other flags, `auto-removeinclude(s)` seems better.

I was also wondering about that. Maybe those should be changed instead? I'm not 
sure, but I think we're slowly transitioning option names to use hyphens just 
like we do with config names? I don't care much, so let me know if you'd still 
prefer to drop the second hyphen.

> pulkit wrote in narrowcommands.py:332
> I don't have a strong opinion on having or dropping EXPERIMENTAL, but we 
> should have some help text. It can be added under verbose section.

I decided to drop EXPERIMENTAL.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6848/new/

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

To: martinvonz, durin42, #hg-reviewers
Cc: pulkit, rdamazio, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to