Task #2591 (closed)
Run pylint/pychecker on all python files
| Reported by: | jamoore | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 5.1.1 |
| Component: | General | Version: | 4.4.8 |
| Keywords: | n.a. | Cc: | python-team@… |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description (last modified by jmoore)
A hudson build should run pylint (flakes, and or checker) on our python files similar to findbugs. This could easily be hooked into the findbugs target on hudson.
The "violations" plugin already can scan pylint, so this would be a good place to start.
See #4749 for one critical use of pylint. The message W0102 (Dangerous default value) should be made an error (rather than just a warning)
Change History (19)
comment:1 Changed 13 years ago by cxallan
comment:2 Changed 13 years ago by jmoore
- Description modified (diff)
- Priority changed from minor to major
comment:3 Changed 12 years ago by jmoore
- Milestone changed from Unscheduled to OMERO-Beta4.4
- Sprint set to 2012-01-17 (6)
Moving to sprint 6 (the bug sprint) for possible handling.
comment:4 Changed 12 years ago by jburel
- Sprint changed from 2012-01-17 (6) to 2012-01-31 (7)
Moved from sprint 2012-01-17 (6)
comment:5 Changed 12 years ago by jmoore
- Sprint changed from 2012-01-31 (7) to 2012-02-14 (8)
Moved from sprint 2012-01-31 (7)
comment:6 Changed 12 years ago by jburel
- Sprint changed from 2012-02-14 (8) to 2012-02-28 (9)
Moved from sprint 2012-02-14 (8)
comment:7 Changed 12 years ago by jburel
- Sprint changed from 2012-02-28 (9) to 2012-03-13 (10)
Moved from sprint 2012-02-28 (9)
comment:8 Changed 12 years ago by jburel
- Sprint changed from 2012-03-13 (10) to 2012-03-27 (11)
Moved from sprint 2012-03-13 (10)
comment:9 Changed 12 years ago by jburel
- Sprint changed from 2012-03-27 (11) to 2012-04-10 (12)
Moved from sprint 2012-03-27 (11)
comment:10 Changed 12 years ago by jburel
- Sprint changed from 2012-04-10 (12) to 2012-04-24 (13)
Moved from sprint 2012-04-10 (12)
comment:11 Changed 12 years ago by jburel
- Sprint changed from 2012-04-24 (13) to 2012-05-08 (14)
Moved from sprint 2012-04-24 (13)
comment:12 Changed 12 years ago by jburel
- Sprint changed from 2012-05-08 (14) to 2012-05-22 (15)
Moved from sprint 2012-05-08 (14)
comment:13 Changed 12 years ago by jburel
- Sprint changed from 2012-05-22 (15) to 2012-06-05 (16)
Moved from sprint 2012-05-22 (15)
comment:14 Changed 12 years ago by jburel
- Sprint changed from 2012-06-05 (16) to 2012-06-19 (17)
Moved from sprint 2012-06-05 (16)
comment:15 Changed 12 years ago by jmoore
- Milestone changed from OMERO-Beta4.4 to OMERO-Beta4.4.1
- Sprint 2012-06-19 (17) deleted
Something we could do for the 4.4 series post release of 4.4.0. Pushing.
comment:16 Changed 12 years ago by jmoore
Referencing ticket #1736 has changed sprint.
comment:17 Changed 11 years ago by sbesson
- Cc python-team@… added; cneves atarkowska cblackburn removed
- Version set to 4.4.8
See https://github.com/openmicroscopy/openmicroscopy/pull/1578 for a first implementation of this task on the CLI plugins on the dev_4_4 branch
comment:18 Changed 9 years ago by cblackburn
- Resolution set to fixed
- Status changed from new to closed
The entire Python code base is now flake8ed via CI.
comment:19 Changed 9 years ago by jamoore
- Milestone changed from 5.x to 5.1.1
Referencing ticket #1736 has changed sprint.