Web Inspector: eslint configuration should be stored as .eslintrc

• Component: Web Inspector
• Status: RESOLVED FIXED

[Jonathan Wells]

An .eslintrc file will allow eslint, run as standalone or in an editor, to automatically pick up style rules agreed upon for the WebKit project. This will define all the allowed globals (WebInspector for instance) and syntax rules for proper inspector JavaScript style. 

We should start with some basics and grow from there.

For more information:
ESLint configuration overview: http://eslint.org/docs/configuring/
ESLint full list of available rules: http://eslint.org/docs/rules/

[Radar WebKit Bug Importer]

<rdar://problem/18012970>

[Jonathan Wells]

Created attachment 236565 [details]
[PATCH] adding .eslintrc

[Joseph Pecoraro]

Comment on attachment 236565 [details]
[PATCH] adding .eslintrc

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

I think this is a reasonable starting point. This will be fun to extend!

> Source/WebInspectorUI/.eslintrc:38
> +    "globals": {
> +        "ConsoleAgent": true,
> +        "DOMAgent": true,
> +        "InspectorBackend": true,
> +        "InspectorFrontendHost": true,
> +        "PageAgent": true,
> +        "WebInspector": true
> +    },

All of the agents will need to be defined in here:

    shell> cd Source
    shell> find . | egrep 'protocol\/.*?json$' | xargs basename | sed -e 's/\.json/Agent/'
    ConsoleAgent
    DebuggerAgent
    GenericTypesAgent # Note we this is a supplemental file so you don't want it.
    InspectorDomainAgent # Note this would just be "InspectorAgent".
    ProfilerAgent
    RuntimeAgent
    ApplicationCacheAgent
    CSSAgent
    DatabaseAgent
    DOMAgent
    DOMDebuggerAgent
    DOMStorageAgent
    IndexedDBAgent
    LayerTreeAgent
    NetworkAgent
    PageAgent
    ReplayAgent
    TimelineAgent
    WorkerAgent

And we will want:

    CodeMirror

> Source/WebInspectorUI/.eslintrc:40
> +        "eqeqeq": 0,

Whoa whoa. We certainly want this on! Maybe not inside UserInterface/External for CodeMirror / Esprima.

[Jonathan Wells]

Comment on attachment 236565 [details]
[PATCH] adding .eslintrc

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

>> Source/WebInspectorUI/.eslintrc:40
>> +        "eqeqeq": 0,
> 
> Whoa whoa. We certainly want this on! Maybe not inside UserInterface/External for CodeMirror / Esprima.

I completely agree, but there were instances of == being used I've found. I'll consider those errors and turn this into a 2, and I'll add an .eslintrc file for CodeMirror and Esprima.

[Jonathan Wells]

Created attachment 236569 [details]
[PATCH] adding .eslintrc, review feedback.

[WebKit Commit Bot]

Comment on attachment 236569 [details]
[PATCH] adding .eslintrc, review feedback.

Clearing flags on attachment: 236569

Committed r172547: <http://trac.webkit.org/changeset/172547>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.