1256933 - [ESLint] Fix ESLint issues in devtools/client/webconsole/test/test-bug-597136-external-script-errors.js

Status: RESOLVED INVALID

(DevTools :: Console, defect, P3)

Description

[Lin Clark [:linclark]]

If you haven’t contributed to Firefox before, follow the steps here to set up your environment: https://developer.mozilla.org/en-US/docs/Tools/Contributing#Getting_set_up

Then, automatically configure ESLint to work with the Firefox specific rules by following the instructions here: https://wiki.mozilla.org/DevTools/CodingStandards

Then you can see the issues that need to be fixed by running

> eslint --no-ignore devtools/client/webconsole/test/test-bug-597136-external-script-errors.js

Comment 1

[[:Towkir] Ahmed]

Hi Lin, 
I would like to work on this bug, I have run that command and found 4 errors and 1 warning, 

But I was just wondering the file only contains :
function f() {
  bogus.g();
}

Not sure what it does, Can you please explain ? 
And the errors + warnings are here:

6:1  warning  Definition for rule 'mozilla/import-globals' was not found           mozilla/import-globals
6:1  error    Definition for rule 'mozilla/no-single-arg-cu-import' was not found  mozilla/no-single-arg-cu-import
6:1  error    Use the global form of 'use strict'                                  strict
6:10 error    'f' is defined but never used                                        no-unused-vars
7:3  error    'bogus' is not defined                                               no-undef

Comment 2

[Lin Clark [:linclark]]

Good questions. I think we can skip this file. It's just being pulled up in a test so the code style doesn't really matter.