Closed
Bug 883100
Opened 12 years ago
Closed 12 years ago
Lint the code and create a make lint target
Categories
(L20n :: JS Library, defect, P4)
Tracking
(Not tracked)
RESOLVED
FIXED
1.0
People
(Reporter: stas, Assigned: stas)
Details
Attachments
(1 file)
|
27.36 KB,
patch
|
zbraniecki
:
review+
|
Details | Diff | Splinter Review |
It would good to use a linter to keep the coding style consistent and avoid stupid mistakes.
|
Assignee | |
Updated•12 years ago
|
Priority: -- → P4
Target Milestone: --- → 1.0
|
|
Assignee | |
Comment 1•12 years ago
|
||
This patch adds a sneaky pre-commit hook which won't allow a commit touching files in bindings and lib. I exclude lib/l20n/intl.js because it's largely not our code.
|
||
Updated•12 years ago
|
Attachment #767107 -
Flags: review?(gandalf) → review+
|
|
Assignee | |
Comment 2•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•