[PATCH] PEP 8: Fix an alignment issue in src/kimchi/auth.py

According to PEP 8 [1], continuation lines should align wrapped elements vertically. [1] http://www.python.org/dev/peps/pep-0008/#indentation Signed-off-by: Zhou Zheng Sheng <zhshzhou@linux.vnet.ibm.com> --- src/kimchi/auth.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py index f38d3d6..242fdcf 100644 --- a/src/kimchi/auth.py +++ b/src/kimchi/auth.py @@ -52,7 +52,7 @@ def authenticate(username, password, service="passwd"): resp.append((password, 0)) elif qtype == PAM.PAM_PROMPT_ERROR_MSG: cherrypy.log.error_log.error("PAM authenticate prompt error " - "message: %s" % query) + "message: %s" % query) resp.append(('', 0)) elif qtype == PAM.PAM_PROMPT_TEXT_INFO: resp.append(('', 0)) -- 1.7.11.7

On Tue 24 Dec 2013 10:21:42 AM CST, Zhou Zheng Sheng wrote:
According to PEP 8 [1], continuation lines should align wrapped elements vertically.
[1] http://www.python.org/dev/peps/pep-0008/#indentation
Signed-off-by: Zhou Zheng Sheng <zhshzhou@linux.vnet.ibm.com> --- src/kimchi/auth.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/kimchi/auth.py b/src/kimchi/auth.py index f38d3d6..242fdcf 100644 --- a/src/kimchi/auth.py +++ b/src/kimchi/auth.py @@ -52,7 +52,7 @@ def authenticate(username, password, service="passwd"): resp.append((password, 0)) elif qtype == PAM.PAM_PROMPT_ERROR_MSG: cherrypy.log.error_log.error("PAM authenticate prompt error " - "message: %s" % query) + "message: %s" % query) resp.append(('', 0)) elif qtype == PAM.PAM_PROMPT_TEXT_INFO: resp.append(('', 0))
Reviewed-by: Mark Wu<wudxw@linux.vnet.ibm.com>

Applied. Thanks. Regards, Aline Manera
participants (4)
-
Aline Manera
-
Mark Wu
-
Ramon Medeiros
-
Zhou Zheng Sheng