commit 678e786c5bb7fca7ac56d3cb280477252ab5a073
Author: Don Penney <email address hidden>
Date: Wed Dec 4 22:26:52 2019 -0500
Address python3 pylint errors and warnings
This commit addresses issues detected by the updated python3 pylint:
- Added a return code to the report_app_dependencies function to
satisfy the E1111 error reported.
- Added line-specific pylint disable for unused-argument for cases
where the inclusion of such arguments in the function signature was
intentional.
- Added line-specific pylint disable for the duplicate-except case
found, as python3 has merged IOError into OSError, while these are
separate exceptions in python2. Once we're running solely on python3,
this duplicate exception handling can be dropped.
Change-Id: I96a521288e71948f06ad0c88a12c8f475ed8bc99
Closes-Bug: 1855180
Signed-off-by: Don Penney <email address hidden>
Reviewed: https:/ /review. opendev. org/697408 /git.openstack. org/cgit/ starlingx/ update/ commit/ ?id=678e786c5bb 7fca7ac56d3cb28 0477252ab5a073
Committed: https:/
Submitter: Zuul
Branch: master
commit 678e786c5bb7fca 7ac56d3cb280477 252ab5a073
Author: Don Penney <email address hidden>
Date: Wed Dec 4 22:26:52 2019 -0500
Address python3 pylint errors and warnings
This commit addresses issues detected by the updated python3 pylint: app_dependencie s function to
- Added a return code to the report_
satisfy the E1111 error reported.
- Added line-specific pylint disable for unused-argument for cases
where the inclusion of such arguments in the function signature was
intentional.
- Added line-specific pylint disable for the duplicate-except case
found, as python3 has merged IOError into OSError, while these are
separate exceptions in python2. Once we're running solely on python3,
this duplicate exception handling can be dropped.
Change-Id: I96a521288e7194 8f06ad0c88a12c8 f475ed8bc99
Closes-Bug: 1855180
Signed-off-by: Don Penney <email address hidden>