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

Mark Wu wudxw at linux.vnet.ibm.com
Tue Dec 24 02:50:11 UTC 2013


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 at 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 at linux.vnet.ibm.com>




More information about the Kimchi-devel mailing list