Closed Bug 1502171 (AS-lint-in-mc) Opened 7 years ago Closed 6 years ago

turning on linting of Activity-Stream code in mozilla-central

Categories

(Firefox :: Messaging System, enhancement, P3)

Product:
Firefox
Component:
Messaging System
Type:
enhancement

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1553917

People

(Reporter: dmosedale, Unassigned)

Details

User Story

As a mozilla-central developer, any changes I make to browser/components/newtab should be linted like other parts of m-c, so that key style & defect types are caught/fixed more efficiently.
After eslint gets vendored into mozilla-central, we'll want to land our own specific eslint plugins there as well, so that we can then remove activity-stream from the top-level .eslintignore and have it get first-class linting support in-tree. There was some discussion about wanting in-tree unit-testing as a prerequisite, so that broad changes are less-likely to introduce issues that should have been caught.
Iteration: --- → 65.2 (Nov 16)
Priority: -- → P2
Iteration: 65.2 (Nov 16) → 65.3 (Nov 30)
Iteration: 65.3 (Nov 30) → 65.4 - Dec 3-9
Iteration: 65.4 - Dec 3-9 → ---
Priority: P2 → P3
Component: Activity Streams: Newtab → Messaging System
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
No longer blocks: as-build-meta
No longer depends on: vendor-eslint
You need to log in before you can comment on or make changes to this bug.