Closed Bug 1015788 Opened 8 years ago Closed 7 years ago

create flake8 lint script and precommit lint hook

Categories

(Input Graveyard :: Code Quality, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: willkg, Assigned: willkg)

References

Details

(Whiteboard: u=dev c=codequality p=2 s=input.2014q3)

We've got a bunch of code and the code quality is ok, but probably not great. It'd be nice to have a script (and possibly a git pre-commit hook) that runs the code (or changes) through a bunch of linting for style issues so that our style can remain consistent without a lot of effort or back-and-forth in PRs.

This bug covers:

1. setting up such things (maybe re-use what Kitsune has?)
2. going through the existing code and fixing style issues
Whiteboard: u=dev c=codequality p= s=input.2014q2
Pushing this off to 2014q3.
Whiteboard: u=dev c=codequality p= s=input.2014q2 → u=dev c=codequality p= s=input.2014q3
The rest of the cosmetic changes from linting and a script and a pre-commit hook all in PR: https://github.com/mozilla/fjord/pull/339
This is still missing baked (or tariff) and jshint linting.
Assignee: nobody → willkg
Whiteboard: u=dev c=codequality p= s=input.2014q3 → u=dev c=codequality p=2 s=input.2014q3
Status: NEW → ASSIGNED
Whiteboard: u=dev c=codequality p=2 s=input.2014q3 → u=dev c=codequality p=2 s=input.2014q4
I just bumped this to 2014q4, but then I thought better of that. Instead, I'm going to reduce the scope of this to just the flake8 stuff and create new bugs for tariff and jshint.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: u=dev c=codequality p=2 s=input.2014q4 → u=dev c=codequality p=2 s=input.2014q3
Summary: flake8, baked (or tariff) and jshint → create flake8 lint script and precommit lint hook
Product: Input → Input Graveyard
You need to log in before you can comment on or make changes to this bug.