Reviewed: https://review.openstack.org/186388 Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=7344fcf2a793069d78055a0bc09a0738d06e361e Submitter: Jenkins Branch: master
commit 7344fcf2a793069d78055a0bc09a0738d06e361e Author: Accela Zhao <email address hidden> Date: Fri May 29 22:18:48 2015 +0800
Fix continuation line under/over indented problems
When you are writing a multiple method invocation (or other similar syntax)
``` obj.method(argument1, argument2, ...) ```
If you put argument1 in the first line, the following lines should have the same hanging distance to argument1.
If you put argument1 in the second line, the hanging distance should always be four.
``` obj.method( argument1, argument2, ... ) ```
Otherwise PEP8 would complain for continuation line under/over indented problems. See below for more details
https://www.python.org/dev/peps/pep-0008/#indentation
This patch fixes all the continuation line indentation problems and enables related checks in tox.ini.
Closes-bug: #1459514 Change-Id: I80995c2ae71b817ad78a87ab7c567f0d36112710
Reviewed: https:/ /review. openstack. org/186388 /git.openstack. org/cgit/ openstack/ magnum/ commit/ ?id=7344fcf2a79 3069d78055a0bc0 9a0738d06e361e
Committed: https:/
Submitter: Jenkins
Branch: master
commit 7344fcf2a793069 d78055a0bc09a07 38d06e361e
Author: Accela Zhao <email address hidden>
Date: Fri May 29 22:18:48 2015 +0800
Fix continuation line under/over indented problems
When you are writing a multiple method invocation (or other similar
syntax)
``` method( argument1,
argument2,
obj.
...)
```
If you put argument1 in the first line, the following lines should have
the same hanging distance to argument1.
If you put argument1 in the second line, the hanging distance should
always be four.
```
obj.method(
argument1,
argument2,
...
)
```
Otherwise PEP8 would complain for continuation line under/over indented
problems. See below for more details
https:/ /www.python. org/dev/ peps/pep- 0008/#indentati on
This patch fixes all the continuation line indentation problems and
enables related checks in tox.ini.
Closes-bug: #1459514 7ad78a87ab7c567 f0d36112710
Change-Id: I80995c2ae71b81