Differential Revision: https://phabricator.services.mozilla.com/D178532

Differential Revision: https://phabricator.services.mozilla.com/D177026

Also keeps 'arrowParens' as 'avoid', rather than switching to the new Prettier default of 'always'.

Differential Revision: https://phabricator.services.mozilla.com/D177018

Differential Revision: https://phabricator.services.mozilla.com/D177477

Bug 1677562 - Remove eslint-plugin-prettier now that Prettier and ESLint are separated. r=Gijs,devtools-reviewers,ochameau

Depends on D175769

Differential Revision: https://phabricator.services.mozilla.com/D175770

Bug 1801591 - Add eslint-plugin-eslint-plugin for linting our ESLint plugin rules, and upgrade most of node_modules. r=mossop

Depends on D168235

Differential Revision: https://phabricator.services.mozilla.com/D168236

Differential Revision: https://phabricator.services.mozilla.com/D163793

Differential Revision: https://phabricator.services.mozilla.com/D158150

Bug 1709150 - Consider linter rule to make https the default for all kind of tests we add. r=Standard8,Gijs

Differential Revision: https://phabricator.services.mozilla.com/D149394

Bug 1766268 - Update eslint-plugin-mozilla's version of estraverse to fix issues with private identifiers. r=mossop

Due to babel still requiring an old version of estraverse, this disables our setup checking for changes
to modules in the eslint-plugin-mozilla package.json. For now, we'll need to remember to ensure there
is a top-level update to go with any changes - though that normally happens anyway.

For good measure, this also updates some of the top-level modules to ensure that node_modules is re-installed for everyone.

Differential Revision: https://phabricator.services.mozilla.com/D144758

Differential Revision: https://phabricator.services.mozilla.com/D144002

Differential Revision: https://phabricator.services.mozilla.com/D140876

Differential Revision: https://phabricator.services.mozilla.com/D137872

Differential Revision: https://phabricator.services.mozilla.com/D128988

Bug 1725934 - Remove unnecessary babel plugin, and update node modules to the latest versions. r=Mardak

plugin-syntax-class-properties was supported by default in Babel 7.14.0.
This also adds top level await support in modules with the upgrade to Babel 7.15.0.

Differential Revision: https://phabricator.services.mozilla.com/D122865

@babel/transform-flow-strip-types and eslint-plugin-flowtype are no longer required as devtools removed the uses of them in bug 1690742.

Differential Revision: https://phabricator.services.mozilla.com/D115402

Differential Revision: https://phabricator.services.mozilla.com/D112345

Bug 1498604 - Make './mach doc' use jsdoc installed into node_modules rather than the system. r=mossop,ahal

Differential Revision: https://phabricator.services.mozilla.com/D111652

CLOSED TREE

Bug 1498604 - Make './mach doc' use jsdoc installed into node_modules rather than the system. r=mossop,ahal

Differential Revision: https://phabricator.services.mozilla.com/D111652

Bug 1702166 - Update ESLint, Babel and associated modules to the latest versions. r=mossop,nchevobbe

Differential Revision: https://phabricator.services.mozilla.com/D110606

Bug 1668274 - update eslint, eslint-plugin-no-unsanitized and mocha to allow use of logical assignment operators, r=Standard8

Differential Revision: https://phabricator.services.mozilla.com/D91933

Currently the `jest/no-standalone-expect` rule needs to be disabled for `jest-in-case` cases blocks as it is not compatible.

Differential Revision: https://phabricator.services.mozilla.com/D87184

Differential Revision: https://phabricator.services.mozilla.com/D84820

Bug 1622717: When parsing an AST allow the chosen parser to override the default scope manager and visitor keys. r=Standard8

There appears to be a bug in babel-eslint where it still uses the old
`ExperimentalSpreadProperty` and `ExperimentalRestProperty` node types when
babel itself (and hence the AST visitor keys) have switched to `SpreadElement`
and `RestElement`. I suspect there isn't much point in filing a babel-eslint
issue for this, they are currently in the process of working on v11 which does
fix the issue but is currently only available as a beta release.

There are three alternatives:

1. Use the beta babel-eslint.
2. Hack the visitor keys to include the old node types.
3. Traverse the AST changing the nodes from the old types to the new types.

Using a beta version is probably not a great idea so this patch does 2 which is
the simplest of the other options. It does have the drawback however that if
any eslint rules we use rely on the new node type names they will fail to work.

Differential Revision: https://phabricator.services.mozilla.com/D66998

--HG--
extra : moz-landing-system : lando

Bug 1617959: Switch to the babel-eslint parser and turn on support for optional chaining and nullish coalescing operator syntaxes. r=Standard8

The babel-eslint parser defaults to assuming all scripts are modules. Although
we're moving in that direction it seems reasonable to set the default as regular
scripts for now. All the places that were previously overriding the parser are
already specifying the sourceType in their eslint configs.

I chose to put the babel config in a file that babel itself won't look to avoid
the risk of this config impacting the use of babel elsewhere in the tree.

Differential Revision: https://phabricator.services.mozilla.com/D66127

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D66129

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D65510

--HG--
extra : moz-landing-system : lando

Bug 1607172 - Update ESLint to the latest version and use ignorePatterns to ignore paths listed in ThirdPartyPaths.txt. r=mossop

Differential Revision: https://phabricator.services.mozilla.com/D59691

--HG--
extra : moz-landing-system : lando

Backed out 9 changesets (bug 1607172, bug 1609998, bug 1608799) for causing xpcshell failures in test_parser.js

CLOSED TREE

Backed out changeset 7753083b67dd (bug 1609998)
Backed out changeset e6f5aac734ab (bug 1608799)
Backed out changeset 90ea35966b73 (bug 1608799)
Backed out changeset 6fafa451b3f9 (bug 1608799)
Backed out changeset e07a4aea2ae1 (bug 1608799)
Backed out changeset d69d6dfdccad (bug 1607172)
Backed out changeset 93023b1b6153 (bug 1607172)
Backed out changeset 99ce7a56080e (bug 1607172)
Backed out changeset 20aa5934c785 (bug 1607172)

Bug 1607172 - Update ESLint to the latest version and use ignorePatterns to ignore paths listed in ThirdPartyPaths.txt. r=mossop

Differential Revision: https://phabricator.services.mozilla.com/D59691

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D57880

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D57880

--HG--
extra : moz-landing-system : lando

Bug 1540982 - Upgrade to ESLint 6.2.2 and switch to ECMA version 11 (BigInt support, Dynamic imports). r=mossop

Differential Revision: https://phabricator.services.mozilla.com/D43590

--HG--
extra : moz-landing-system : lando

This picks up various improvements, especially to how configurations are handled and some new rules.

Differential Revision: https://phabricator.services.mozilla.com/D37271

--HG--
extra : moz-landing-system : lando

Differential Revision: https://phabricator.services.mozilla.com/D38093

--HG--
extra : moz-landing-system : lando

Bug 1494796 - Removing threadClient specifics from DebuggerClient Special case resume; r=jdescottes,jlast

The resume case is much more complex than the other events, because we do an
unsafeSynchronize to send an unsolicited pause. In the old system, the resume response would have
been ignored, but that is no longer the case. With the new system, we do not want to send a response
to a resume action if it did not come from the UI. This also update the debugger panel code to
accept a resume.

Differential Revision: https://phabricator.services.mozilla.com/D32697

--HG--
extra : moz-landing-system : lando

Backed out changeset 5db908b26d50 (bug 1494796)
Backed out changeset c48f00f0df72 (bug 1494796)
Backed out changeset 591453b88e8b (bug 1494796)
Backed out changeset a14e820311bc (bug 1494796)
Backed out changeset 0e214d450b35 (bug 1494796)
Backed out changeset 1a4ab8b35a85 (bug 1494796)
Backed out changeset fe1559f5f1d4 (bug 1494796)
Backed out changeset 35d967de4223 (bug 1494796)
Backed out changeset 1d21a55cae15 (bug 1494796)
Backed out changeset 33eec873a43e (bug 1494796)
Backed out changeset 25e69c21dc2e (bug 1494796)
Backed out changeset b900d41c8ae8 (bug 1494796)
Backed out changeset c2a034e34fa6 (bug 1494796)

Bug 1494796 - Removing threadClient specifics from DebuggerClient Special case resume; r=jdescottes,jlast

The resume case is much more complex than the other events, because we do an
unsafeSynchronize to send an unsolicited pause. In the old system, the resume response would have
been ignored, but that is no longer the case. With the new system, we do not want to send a response
to a resume action if it did not come from the UI. This also update the debugger panel code to
accept a resume.

Differential Revision: https://phabricator.services.mozilla.com/D32697

--HG--
extra : moz-landing-system : lando