Re-enable pylint CI checker

jerryscript-project / jerryscript

Ultra-lightweight JavaScript engine for the Internet of Things.

https://jerryscript.net

Apache License 2.0

6.87k stars 666 forks source link

Re-enable pylint CI checker #5096

Closed matetokodi closed 8 months ago

matetokodi commented 9 months ago

This patch re-enables the pylint CI checker

Adjust pylintrc to account for new checkers introduced since updating from ubuntu-18.04 to ubuntu-22-04: ignore imports, disable some style-based checks: e.g: consider-using-f-string, no-else-continue, among others.

Update test262-harness.py to use argparse instead of the now deprecated optparse.

Rename variables in jerry_client_main.py that redefined python builtins or shadowed variables from an outer scope

matetokodi commented 9 months ago

Over time this became an overall Python modernization patch, with the following additions:

Use f-strings
Remove from __future__ imports
Remove shebangs from non-executable files
Re-enable all pylint checkers except missing-docstring
Set minimum python version to 3.6