1160489 - Enable eslint rules for Loop: comma related

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect, P3)

Description

[Mark Banner (:standard8)]

Enable some more eslint rules for Loop. Rules to enable in this bug:

* comma-dangle (no commas at end of sections)
* comma-spacing (trailing space after comma)

To enable the rule, remove the line from browser/components/loop/.eslintrc and save it.

Then you can run eslint in the browser/components/loop directory with:

  eslint  --ext .js --ext .jsm --ext .jsx .

The errors listed in the output are to be fixed.

If there's errors in a .js file that has an associated .jsx file, then fix the jsx file and run the react tools to generate the .js file (https://wiki.mozilla.org/Loop/Development#Developing).

If you need more help setting up eslint, see https://wiki.mozilla.org/Loop/Development#Additional_Requirements

Information about the eslint rules can be found here:

http://eslint.org/docs/rules/

Comment 1

[Martin Tomes [:tomesh]]

Hi, I would like to start working on this bug. Can you please assign me?

Comment 2

[Martin Tomes [:tomesh]]

Attachment: rev1 - eslint comma related errors corrected.

Comment 3

[Mark Banner (:standard8)]

Comment on attachment 8602132 [details] [diff] [review]
rev1 - eslint comma related errors corrected.

Review of attachment 8602132 [details] [diff] [review]:
-----------------------------------------------------------------

Great patch! Thanks for that. There's one minor nit, plus an additional couple of changes from what I checked in earlier today.

Given the simplicity of the fixes, I'll just fix them when I land in a couple of mins.

::: browser/components/loop/test/xpcshell/test_loopservice_busy.js
@@ +53,5 @@
>    setupFakeLoopServer();
>  
>    // Setup fake login state so we get FxA requests.
>    const MozLoopServiceInternal = Cu.import("resource:///modules/loop/MozLoopService.jsm", {}).MozLoopServiceInternal;
> +  MozLoopServiceInternal.fxAOAuthTokenData = {token_type:"bearer", access_token:"1bad3e44b12f77a88fe09f016f6a37c42e40f974bc7a8b432bb0d2f0e37e1752", scope:"profile"};

I know its not your fault, but whilst we're here, this line is a bit long, so we should reformat it the way we format the rest of our objects (I think its called json style, but its not quite!):

MozLoopServiceInternal.fxAOAuthTokenData = {
  token_type: "bearer",
  access_token: "1bad3e44b12f77a88fe09f016f6a37c42e40f974bc7a8b432bb0d2f0e37e1752",
  scope: "profile"
};

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/32df49355c68

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/32df49355c68