Code Review / csit.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: de305de) | patch
Fix pylint error in PapiExecutor 33/18833/2
authorVratko Polak <vrpolak@cisco.com>
Fri, 12 Apr 2019 11:22:00 +0000 (13:22 +0200)
committerTibor Frank <tifrank@cisco.com>
Mon, 15 Apr 2019 07:24:47 +0000 (07:24 +0000)
commit5899ac744eb5f722f53151c98e5acfdbdf678d0c
tree94b3d0c2d0ba343bcd4ee3422724e76757ccfe75tree | snapshot
parentde305de0b89f083037bdac86bb8f3739f5eeecf9commit | diff
Fix pylint error in PapiExecutor

+ Remove ret_val as it cannot be nonzero outside _execute_papi.
+ Change indentation to save vertical space.
+ Return computed values without storing them to variable.
+ Add TODO to remove duplicated method.
+ Add missing fields to docstring.
+ Minor formatting changes.

Change-Id: I8e8f41d139de8aff3b76a7fe27ee450e89e8d1de
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
resources/libraries/python/PapiExecutor.py diff | blob | history
Integration tests