[Webkit-unassigned] [Bug 143212] Web Inspector: Add more ESLint rules that reflect the current state of the code base

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=143212

--- Comment #8 from tobi+bugzilla at basecode.de ---
Comment on attachment 249712
  --> https://bugs.webkit.org/attachment.cgi?id=249712
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=249712&action=review

>>>> Source/WebInspectorUI/.eslintrc:83
>>>> +        "comma-dangle": 0,
>>> 
>>> I don't like using a comma dangle.
>> 
>> The rule is turned off. If you don't want contributors to use trailing commas than we could set "comma-dangle" to "never".
> 
> I do! It makes diffs easier to read. Also, any good minifier can get rid of the extra commas.

ESLint offers 3 variations. "never", "always" and "always-multiline". https://github.com/eslint/eslint/blob/master/docs/rules/comma-dangle.md.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150330/82a7c228/attachment-0002.html>