Opened 3 years ago

Closed 3 years ago

#4993 closed change (fixed)

Apply the operator-linebreak ESLint rule consistently for all operators

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

https://codereview.adblockplus.org/29384022

Description

Background

Currently our ESLint configuration uses the default settings for the operator-linebreak rule which turns out to be this:

"after", { "overrides": { "?": "before", ":": "before" } }

What to change

We want to apply the rule consistently for all operators, so our rule should look like this:

"operator-linebreak": ["error", "after"]

Change History (4)

comment:1 Changed 3 years ago by kzar

  • Cc greiner added

comment:3 Changed 3 years ago by abpbot

comment:4 Changed 3 years ago by kzar

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.