On Tue, Aug 30, 2016 at 08:58:09PM -0700, Sean Farley wrote:
> # HG changeset patch
> # User Sean Farley <s...@farley.io>
> # Date 1472591416 25200
> #      Tue Aug 30 14:10:16 2016 -0700
> # Node ID a6f4b32f655ba895875640534b051c5ebe632695
> # Parent  78dd9f1db1b1cf5a869e1240bb4ed523e930c907
> # EXP-Topic flake8
> flake8: fix E127 style

I might suggest describing what e127 is in the log message? This kind
of buries the lede in terms of actual content changes IMO.

>
> diff --git a/hgext3rd/topic/topicmap.py b/hgext3rd/topic/topicmap.py
> --- a/hgext3rd/topic/topicmap.py
> +++ b/hgext3rd/topic/topicmap.py
> @@ -125,11 +125,11 @@ class topiccache(oldbranchcache):
>              if (self.tipnode == repo.changelog.node(self.tiprev)):
>                  fh = scmutil.filteredhash(repo, self.tiprev)
>                  if fh is None:
>                      fh = nullid
>                  if ((self.filteredhash == fh)
> -                     and (self.phaseshash == _phaseshash(repo, 
> self.tiprev))):
> +                    and (self.phaseshash == _phaseshash(repo, self.tiprev))):
>                      return True
>              return False
>          except IndexError:
>              return False
>
> diff --git a/setup.cfg b/setup.cfg
> --- a/setup.cfg
> +++ b/setup.cfg
> @@ -1,2 +1,2 @@
>  [flake8]
> -ignore = E261, E266, E302, E129, E731, E124, E713, E301, E501, E111, E123, 
> E222, W503, W391, E127
> +ignore = E261, E266, E302, E129, E731, E124, E713, E301, E501, E111, E123, 
> E222, W503, W391
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel@mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to