Closed Bug 1467100 Opened 6 years ago Closed 6 years ago

Remove duplicated eslint rules

Categories

(Remote Protocol :: Marionette, enhancement)

enhancement
Not set
normal

Tracking

(firefox62 fixed)

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: ato, Assigned: ato)

Details

Attachments

(1 file)

It looks like the default eslint rules in central, located at
tools/lint/eslint/eslint-plugin-mozilla/lib/configs/recommended.js, have
adopted a few of the more strict rules we have in the Marionette
eslint specialisation at testing/marionette/.eslintrc.js.

These are:

  * no-new-object
  * no-unused-vars
  * semi

We can remove these from the Marionette eslint config since they
are duplicates.
Assignee: nobody → ato
Status: NEW → ASSIGNED
Attachment #8983745 - Flags: review?(hskupin)
Comment on attachment 8983745 [details]
Bug 1467100 - Remove duplicate eslint rules for testing/marionette

https://reviewboard.mozilla.org/r/249580/#review255788
Attachment #8983745 - Flags: review?(hskupin) → review+
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3af85b19ae18
Remove duplicate eslint rules for testing/marionette r=whimboo
https://hg.mozilla.org/mozilla-central/rev/3af85b19ae18
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: