On Wed, 05 Dec 2018 08:23:17 -0500, Yuya Nishihara <y...@tcha.org> wrote:

On Tue, 4 Dec 2018 10:06:24 -0500, Augie Fackler wrote:

So, options to move forward:
1) blacken everything (controversial for good reasons)
2) try black only on a subset
3) explore yapf
4) Give up and keep manually formatting files (I'd rather not do this, but I understand if it's where we end up)

My vote: 3 > 4 > 2 > 1

I'm not super enthusiastic about 100%-machine-forced formatting. I like
consistency level provided by e.g. astyle command. clang-format is pretty
good IMHO, but the black seems to sacrifice the code readability.

+1.

That said, I got used to longnamesthataresmooshedtogether, so I can probably adjust to anything after awhile.
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to