Project

General

Profile

Actions

Architecture #12636

closed

Use pylint to verify python code

Added by François ARMAND almost 6 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
N/A
Category:
Packaging
Target version:
Effort required:
Name check:
Fix check:
Regression:

Description

We have a ton of problems due to python lack of static checks. We have typos (), we have missing imports (#12633), and this has dire consequences in production.

For example, #12633 could have been avoided without any special configuration:

% sudo pacman -S python-pylint
...
% pylint --disable=I,R,C,W ncf                                                                                      [18-05-16 11:16:19]
No config file found, using default configuration
************* Module ncf
E: 62,87: Using variable 'technique' before assignment (used-before-assignment)
E: 71,23: Class 'traceback' has no 'format_exc' member (no-member)
E: 71,23: Undefined variable 'traceback' (undefined-variable)

------------------------------------------------------------------
Your code has been rated at 7.79/10 (previous run: 7.79/10, +0.00)

We need to force use pylint in any python code before a release. The best place seems to be in CI. The ticket project should be changed if needed, depending on the implemented solution.


Subtasks 8 (0 open8 closed)

Architecture #12639: Use pylint to verify python code in ncfReleasedBenoît PECCATTEActions
Architecture #12640: Fix pylint error in ncf 4.1ReleasedBenoît PECCATTEActions
Architecture #12651: Fix pylint error in ncf 4.3ReleasedVincent MEMBRÉActions
Architecture #12641: Use pylint to verify python code in relayReleasedBenoît PECCATTEActions
Architecture #12642: Fix pylint error in relay codeReleasedVincent MEMBRÉActions
Architecture #12652: Fix pylint error in relay codeReleasedVincent MEMBRÉActions
Architecture #12655: Use pylint to verify python code in rudder repoReleasedVincent MEMBRÉActions
Architecture #12656: Use pylint to verify python code in rudder-pluginsReleasedBenoît PECCATTEActions
Actions #1

Updated by Benoît PECCATTE almost 6 years ago

  • Status changed from New to In progress
Actions #2

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.12 to 4.1.13
Actions #3

Updated by Benoît PECCATTE almost 6 years ago

  • Target version changed from 4.1.13 to 411
Actions #4

Updated by Benoît PECCATTE almost 6 years ago

  • Target version changed from 411 to 4.1.13
Actions #5

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.13 to 4.1.14
Actions #6

Updated by Benoît PECCATTE over 5 years ago

  • Target version changed from 4.1.14 to 4.1.15
Actions #7

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions #8

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.16 to 4.1.17
Actions #9

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.17 to 4.1.18
Actions #10

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.18 to 4.1.19
Actions #11

Updated by Alexis Mousset about 5 years ago

  • Target version changed from 4.1.19 to 4.1.20
Actions #12

Updated by François ARMAND about 5 years ago

  • Target version changed from 4.1.20 to 4.1.21
Actions #13

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 4.1.21 to 4.1.22
Actions #14

Updated by Benoît PECCATTE almost 5 years ago

  • Target version changed from 4.1.22 to 5.0.10
Actions #15

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 5.0.10 to 5.0.11
Actions #16

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 5.0.11 to 5.0.12
Actions #17

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 5.0.12 to 5.0.13
Actions #18

Updated by Benoît PECCATTE over 4 years ago

  • Status changed from In progress to Resolved

Done in sub tasks

Actions

Also available in: Atom PDF