Closed Bug 1169939 Opened 9 years ago Closed 9 years ago

Use less noisy pylint/landscape.io configs

Categories

(Tree Management :: Treeherder, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

To help with bug 1169938, we should:
(a) check a pylintrc into the repo with sensible defaults (since otherwise we just use landscape.io's defaults - and also have no way to repro locally)
(b) tweak .landscape.yaml to reduce the number of false positives (eg turn off warnings for tests, at least initially)
Attachment #8613240 - Flags: review?(cdawson)
Attachment #8613240 - Flags: review?(cdawson) → review+
Commits pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/7ccb6b2d6a2272845786948b90d5c4f3dd80d5e2
Bug 1169939 - Switch .landscape.yaml to 2-space indent

https://github.com/mozilla/treeherder/commit/16ef326225935dc35332baeea27593d5153fc8c9
Bug 1169939 - Tweak .landscape.yaml to improve signal to noise ratio

* Turns off warnings for tests (there are particularly noisy due to
  mocking - we can always turn this back on later).
* Ignores some files/folders that are either for third party code, or
  else don't play nicely with pylint/landscape.io (eg update.py, since
  commander isn't installed).
* Adds the prod.txt requirements file, so landscape.io can resolve the
  New Relic package references.

https://github.com/mozilla/treeherder/commit/e60a96bdf29c2eab9ea5927906b01b4aa2e07f5a
Bug 1169939 - Add a first pass at a pylintrc to the repo

Uses many of the defaults from prospector (used by landscape.io):
https://github.com/landscapeio/prospector/tree/master/prospector/profiles/profiles

...particularly those in `strictness_{veryhigh,high,medium}.yaml`

landscape.io will now use this file instead of its own defaults.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Component: Treeherder: Docs & Development → TreeHerder
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: