Skip to content

Commit

Permalink
Bug 1551218 - Part 1: Enable running top-level eslint for the devtool…
Browse files Browse the repository at this point in the history
…s debugger, r=Standard8,jlast

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

--HG--
extra : moz-landing-system : lando
  • Loading branch information
Victor Porof committed May 17, 2019
1 parent de5a1f7 commit e4c6310
Show file tree
Hide file tree
Showing 8 changed files with 233 additions and 39 deletions.
37 changes: 33 additions & 4 deletions .eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -101,12 +101,41 @@ devtools/shared/tests/mochitest/*.html
devtools/shared/webconsole/test/test_*.html
devtools/client/webreplay/mochitest/examples/*.html

# Soon to be removed, the new/ directory is explicitly excluded below due to
# also being an imported repository.
devtools/client/debugger/**
# Ignore devtools debugger files
# Keep in sync with devtools/client/debugger/.eslintignore
devtools/client/debugger/assets/*
devtools/client/debugger/src/test/examples/**
devtools/client/debugger/src/test/integration/**
devtools/client/debugger/src/test/unit-sources/**
devtools/client/debugger/src/**/fixtures/**
devtools/client/debugger/src/test/mochitest/**
devtools/client/debugger/bin/
devtools/client/debugger/packages/**/fixtures/**
devtools/client/debugger/node_modules
devtools/client/debugger/out

# Ignore devtools debugger files
# Keep in sync with devtools/client/debugger/.prettierignore
devtools/client/debugger/src/workers/parser/tests/fixtures/functionNames.js
devtools/client/debugger/src/workers/parser/tests/fixtures/scopes/*.js
devtools/client/debugger/src/workers/parser/tests/fixtures/pause/*.js
devtools/client/debugger/src/test/mochitest/examples/babel/polyfill-bundle.js
devtools/client/debugger/src/test/mochitest/examples/babel/fixtures/*/input.js
devtools/client/debugger/src/test/mochitest/examples/babel/fixtures/*/output.js
devtools/client/debugger/src/test/mochitest/examples/babel/fixtures/*/output.js.map
devtools/client/debugger/src/test/mochitest/examples/ember/quickstart

# Ignore devtools debugger files which aren't intended for linting, and also
# aren't included in any .eslintignore or .prettierignore file.
# See https://github.com/firefox-devtools/debugger/blob/master/package.json#L24
devtools/client/debugger/configs/**
devtools/client/debugger/dist/**
devtools/client/debugger/flow-typed/**
devtools/client/debugger/images/**
devtools/client/debugger/test/**
devtools/client/debugger/index.html

# Ignore devtools imported repositories
devtools/client/debugger/**
devtools/client/shared/components/reps/**

# Ignore devtools preferences files
Expand Down
2 changes: 2 additions & 0 deletions devtools/client/debugger/.eslintignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
# Keep in sync with top-level .eslintignore

assets/*
src/test/examples/**
src/test/integration/**
Expand Down
2 changes: 2 additions & 0 deletions devtools/client/debugger/.prettierignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
# Keep in sync with top-level .eslintignore

src/workers/parser/tests/fixtures/functionNames.js
src/workers/parser/tests/fixtures/scopes/*.js
src/workers/parser/tests/fixtures/pause/*.js
Expand Down
1 change: 0 additions & 1 deletion devtools/client/debugger/test/mochitest/.eslintrc
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@
"assertEmptyLines": false,
"assertPaused": false,
"assertPausedLocation": false,
"assertPreviewTextValue": false,
"assertHighlightLocation": false,
"assertScopes": false,
"createDebuggerContext": false,
Expand Down
Loading

0 comments on commit e4c6310

Please sign in to comment.