D905: style: never put multiple statements on one line

indygreg (Gregory Szorc) phabricator at mercurial-scm.org
Mon Oct 2 13:25:54 EDT 2017


indygreg added subscribers: sid0, indygreg.
indygreg accepted this revision.
indygreg added a comment.
This revision is now accepted and ready to land.


  Since you weren't at the Sprint, I think the decision for style is that we'd check the existing wiki page for the style guide into the repo (possibly along with profiles/configs for various linting/reformatting tools). Then, future changes to style would be handled via code review.
  
  @sid0's annotation revision skipping feature is maturing and combined with @yuya's patches to resolve a revset from a file will allow us to check in a file containing a list of "reformatting" revisions to automatically/easily skip when doing annotate. So we should be more willing to mass rewrite code.

REPOSITORY
  rHG Mercurial

BRANCH
  style-fixes (bookmark) on default (branch)

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

To: alex_gaynor, durin42, #hg-reviewers, ryanmce, indygreg
Cc: indygreg, sid0, lothiraldan, ryanmce, mercurial-devel


More information about the Mercurial-devel mailing list