GabrielBrascher commented on pull request #3730: URL: https://github.com/apache/cloudstack/pull/3730#issuecomment-671755604
Current Travis Pylint errors are related to the fact of Pylint evaluating a, supposedly, Python2 code. ``` pylint --disable=R,C,W configure.py No config file found, using default configuration ************* Module configure E: 24, 0: No name 'request' in module 'urllib' (no-name-in-module) E: 24, 0: Unable to import 'urllib.request' (import-error) E: 25, 0: No name 'parse' in module 'urllib' (no-name-in-module) E: 25, 0: Unable to import 'urllib.parse' (import-error) E: 26, 0: No name 'error' in module 'urllib' (no-name-in-module) E: 26, 0: Unable to import 'urllib.error' (import-error) E: 81,23: Undefined variable 'urllib' (undefined-variable) E: 82,26: Undefined variable 'urllib' (undefined-variable) E: 84,27: Undefined variable 'urlopen' (undefined-variable) ------------------------------------------------------------------- Your code has been rated at 9.39/10 (previous run: 10.00/10, -0.61) ``` If Travis had pylint3, then it would be evaluating a Python3 code. I get the following result on pylint3 ``` pylint3 --disable=R,C,W configure.py No config file found, using default configuration ------------------------------------------------------------------- Your code has been rated at 10.00/10 (previous run: 9.39/10, +0.61) ``` :arrow_down: The same applies for the errors raised by Travis regarding `passwd_server_ip.py`: ``` pylint --disable=R,C,W passwd_server_ip.py No config file found, using default configuration ************* Module passwd_server_ip E: 34, 0: No name 'parse' in module 'urllib' (no-name-in-module) E: 34, 0: Unable to import 'urllib.parse' (import-error) E: 36, 0: Unable to import 'http.server' (import-error) E: 37, 0: Unable to import 'socketserver' (import-error) ------------------------------------------------------------------- Your code has been rated at 8.60/10 (previous run: 10.00/10, -1.40) ``` vs pylint3 ``` pylint3 --disable=R,C,W passwd_server_ip.py No config file found, using default configuration ------------------------------------------------------------------- Your code has been rated at 10.00/10 (previous run: 8.60/10, +1.40) ``` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
