Code Review / csit.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 563dc7e) | patch
Fix pylint error in PapiExecutor 64/18864/1
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:25:05 +0000 (07:25 +0000)
commit7f8cd27748dcf7311ba0ed56b077bcef9fc351f9
tree10f9d97b0e4b8cba3beb9e6ce12cda38ccf80370tree | snapshot
parent563dc7e4cda1894f987ef149567daa6b64583daccommit | 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>
(cherry picked from commit 5899ac744eb5f722f53151c98e5acfdbdf678d0c)
resources/libraries/python/PapiExecutor.py diff | blob | history
Integration tests