Opened on 02/13/2017 at 05:19:09 AM

Closed on 03/15/2017 at 12:04:11 PM

Last modified on 03/30/2017 at 07:00:10 AM

#4878 closed change (fixed)

Start using ESLint for adblockpluscore repository

Reported by: kzar Assignee: kzar
Priority: Unknown Milestone:
Module: Unknown Keywords:
Cc: trev Blocked By: #3692
Blocking: #5061 Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by kzar)

Background

With #3692 we're adding a base ESLint configuration to the buildtools repository. Once that is finished we want to start using the configuration for the adblockpluscore repository.

What to change

  • Add a ESLint configuration to adblockpluscore which inherits from our base configuration:
    {
      "extends": "eslint-config-eyeo",
      "root": true,
      ...
    }
    
  • If necessary add an ESLint ignore file to ignore third party libraries such as jQuery.
  • Make the necessary changes to the adblockpluscore code so that linting passes.
  • Add a note to the README explaining how to lint the code (eslint *.js chrome lib test).

Notes

  • Since we're now using String.prototype.includes we'll need to bump the minimum Firefox version to 40 when we update the adblockpluscore dependency to include these changes.
  • We've changes Cu.import calls to use the alternative syntax, this will require changes in the adblockpluschrome compat code. For example Cu.import("resource://gre/modules/XPCOMUtils.jsm"); becomes const {XPCOMUtils} = Cu.import("resource://gre/modules/XPCOMUtils.jsm", {});.

Attachments (0)

Change History (6)

comment:1 Changed on 02/13/2017 at 06:22:05 AM by kzar

  • Description modified (diff)

comment:2 Changed on 03/07/2017 at 01:47:20 PM by kzar

  • Description modified (diff)

comment:3 Changed on 03/15/2017 at 11:34:52 AM by abpbot

A commit referencing this issue has landed:
Issue 4878 - Start using ESLint for adblockpluscore

comment:4 Changed on 03/15/2017 at 12:04:11 PM by kzar

  • Resolution set to fixed
  • Status changed from new to closed

comment:5 Changed on 03/30/2017 at 05:29:20 AM by kzar

  • Description modified (diff)

comment:6 Changed on 03/30/2017 at 07:00:10 AM by kzar

  • Blocking 5061 added

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from kzar.
 
Note: See TracTickets for help on using tickets.