Fix the worst of the pylint warnings

RESOLVED WONTFIX

Status

Tree Management
Treeherder: Docs & Development
P3
normal
RESOLVED WONTFIX
3 years ago
4 months ago

People

(Reporter: emorley, Unassigned)

Tracking

Details

(Reporter)

Description

3 years ago
If we're going to have landscape.io run on every commit, we might as well use it (or if we find it to not be useful let's switch it off; but let's give it a chance first).

I think much of the advantage of the score based approach of landscape.io - is that we don't have to fix all the warnings - since in theory (there are a few landscape.io bugs - have reported upstream) we can just look at the score to see if a given landing made things worse - even if we weren't at 100% before.

That said, it would be good to drive down some of the existing warnings to make it easier to see new ones as they are introduced. We should also pick some more sensible defaults for file ignore lists, pylint warning type ignore lists etc.
(Reporter)

Updated

3 years ago
Depends on: 1169939
(Reporter)

Updated

2 years ago
Summary: Fix the worst of the pylint/landscape.io warnings → Fix the worst of the pylint warnings
(Reporter)

Updated

4 months ago
Status: NEW → RESOLVED
Last Resolved: 4 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.