Patchwork [07,of,12,topics] flake8: fix E123 style

login
register
mail settings
Submitter Sean Farley
Date Aug. 31, 2016, 3:58 a.m.
Message ID <8eb5de0920f5dce2fbf4.1472615893@laptop.local>
Download mbox | patch
Permalink /patch/16508/
State Accepted
Headers show

Comments

Sean Farley - Aug. 31, 2016, 3:58 a.m.
# HG changeset patch
# User Sean Farley <sean@farley.io>
# Date 1472595476 25200
#      Tue Aug 30 15:17:56 2016 -0700
# Node ID 8eb5de0920f5dce2fbf4d511b73ea6bcc7d6fe8e
# Parent  e32ae663d1dcbcd21bcce74e58ddfdf8dcc7368c
# EXP-Topic flake8
flake8: fix E123 style
Pierre-Yves David - Sept. 6, 2016, 11:42 a.m.
On 08/31/2016 05:58 AM, Sean Farley wrote:
> # HG changeset patch
> # User Sean Farley <sean@farley.io>
> # Date 1472595476 25200
> #      Tue Aug 30 15:17:56 2016 -0700
> # Node ID 8eb5de0920f5dce2fbf4d511b73ea6bcc7d6fe8e
> # Parent  e32ae663d1dcbcd21bcce74e58ddfdf8dcc7368c
> # EXP-Topic flake8
> flake8: fix E123 style

I'm not sure how this one is an improvement, I've ignored is for now.

>
> diff --git a/hgext3rd/topic/__init__.py b/hgext3rd/topic/__init__.py
> --- a/hgext3rd/topic/__init__.py
> +++ b/hgext3rd/topic/__init__.py
> @@ -201,11 +201,11 @@ def reposetup(ui, repo):
>
>  @command('topics [TOPIC]', [
>          ('', 'clear', False, 'clear active topic if any'),
>          ('', 'change', '', 'revset of existing revisions to change topic'),
>          ('l', 'list', False, 'show the stack of changeset in the topic'),
> -    ] + commands.formatteropts)
> +] + commands.formatteropts)
>  def topics(ui, repo, topic='', clear=False, change=None, list=False, **opts):
>      """View current topic, set current topic, or see all topics."""
>      if list:
>          if clear or change:
>              raise error.Abort(_("cannot use --clear or --change with --list"))
> 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
> +ignore = E261, E266, E302, E129, E731, E124, E713, E301, E501, E111
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel@mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
>

Patch

diff --git a/hgext3rd/topic/__init__.py b/hgext3rd/topic/__init__.py
--- a/hgext3rd/topic/__init__.py
+++ b/hgext3rd/topic/__init__.py
@@ -201,11 +201,11 @@  def reposetup(ui, repo):
 
 @command('topics [TOPIC]', [
         ('', 'clear', False, 'clear active topic if any'),
         ('', 'change', '', 'revset of existing revisions to change topic'),
         ('l', 'list', False, 'show the stack of changeset in the topic'),
-    ] + commands.formatteropts)
+] + commands.formatteropts)
 def topics(ui, repo, topic='', clear=False, change=None, list=False, **opts):
     """View current topic, set current topic, or see all topics."""
     if list:
         if clear or change:
             raise error.Abort(_("cannot use --clear or --change with --list"))
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
+ignore = E261, E266, E302, E129, E731, E124, E713, E301, E501, E111