Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-27 06:43:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1830265 - Tidy up .prettierignore. r=Gijs,sync-reviewers,markh

Browse Source 
Enables Prettier for (these don't need to be excluded):
- extensions/pref/autoconfig/test/unit/autoconfig.js
- services/sync/tests/unit/prefs_test_prefs_store.js

Removes duplications where the files are already ignored in the .eslintignore section.

Re-organises the structure of the remaining parts to make it clearer why they are being excluded.

Differential Revision: https://phabricator.services.mozilla.com/D176636
This commit is contained in:
Mark Banner 2023-05-04 17:21:03 +00:00
parent 0260a3c821
commit 2d03564d1e
4 changed files with 13 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.eslintignore
View File

@ -90,6 +90,7 @@ devtools/client/webconsole/test/browser/test-autocomplete-mapped.src.js
devtools/client/inspector/markup/test/shadowdom_open_debugger.min.js
devtools/client/webconsole/test/browser/test-click-function-to-source*.js
devtools/client/webconsole/test/browser/test-external-script-errors.js
devtools/client/webconsole/test/browser/test-mangled-function.*
devtools/client/webconsole/test/browser/test-message-categories-canvas-css.js
devtools/client/webconsole/test/browser/test-message-categories-empty-getelementbyid.js
devtools/client/webconsole/test/browser/test-sourcemap*.js

46
.prettierignore
View File

@ -14,55 +14,24 @@
# This should be re-enabled in bug 1827456.
js/src/builtin
# Don't format README files.
README
# Ignore default pref files which have a .js suffix but aren't parsed normally as JavaScript.
browser/app/profile/channel-prefs.js
browser/app/profile/firefox.js
browser/branding/aurora/pref/firefox-branding.js
browser/branding/nightly/pref/firefox-branding.js
browser/branding/official/pref/firefox-branding.js
browser/branding/unofficial/pref/firefox-branding.js
devtools/client/preferences/debugger.js
extensions/pref/autoconfig/test/unit/autoconfig.js
mobile/android/app/geckoview-prefs.js
mobile/android/app/mobile.js
mobile/android/installer/mobile-l10n.js
modules/libpref/init/all.js
modules/libpref/test/unit/data/testParser.js
modules/libpref/test/unit/data/testPrefLocked.js
modules/libpref/test/unit/data/testPrefSticky.js
modules/libpref/test/unit/extdata/testExt.js
services/sync/tests/unit/prefs_test_prefs_store.js
# Ignore CORS fixtures which require specific resource hashes.
dom/security/test/sri/script*
# Ignore other DOM fixtures with funky chars.
# Files that are intentionally broken for tests.
dom/base/test/file_bug687859-charset.js
dom/base/test/file_bug687859-inherit.js
# Ignore devtools debugger files
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/**
# These are source mapped and the locations are asserted in the test case.
devtools/client/webconsole/test/browser/test-mangled-function.*
# This file is based on another and we want to keep the syntax the same.
devtools/shared/plural-form.js
# Intentionally broken files
parser/htmlparser/tests/mochitest/broken_xml.xhtml
parser/htmlparser/tests/mochitest/test_bug563322.xhtml
# A mainly-imported file, that we don't want to format, but want to keep an eye
# on with the other imported files.
# Files that are specific formats/layouts for tests.
devtools/client/debugger/src/**/fixtures/**
# Imported files where we don't want to change the format, but we do want ESLint
# to keep an eye on them (hence not in Generated.txt/ThirdPartyFiles.txt)
devtools/shared/plural-form.js
toolkit/components/formautofill/shared/CreditCardRuleset.sys.mjs
##############################################################################
@ -148,6 +117,7 @@ devtools/client/webconsole/test/browser/test-autocomplete-mapped.src.js
devtools/client/inspector/markup/test/shadowdom_open_debugger.min.js
devtools/client/webconsole/test/browser/test-click-function-to-source*.js
devtools/client/webconsole/test/browser/test-external-script-errors.js
devtools/client/webconsole/test/browser/test-mangled-function.*
devtools/client/webconsole/test/browser/test-message-categories-canvas-css.js
devtools/client/webconsole/test/browser/test-message-categories-empty-getelementbyid.js
devtools/client/webconsole/test/browser/test-sourcemap*.js

1
extensions/pref/autoconfig/test/unit/autoconfig.js
View File

@ -3,4 +3,3 @@ pref("general.config.sandbox_enabled", true);
pref("general.config.filename", "autoconfig.cfg");
pref("general.config.vendor", "autoconfig");
pref("general.config.obscure_value", 0);

5
services/sync/tests/unit/prefs_test_prefs_store.js
View File

@ -30,7 +30,10 @@ user_pref("testing.dont.change", "Please don't change me.");
user_pref("testing.turned.off", "I won't get synced.");
user_pref("testing.not.turned.on", "I won't get synced either!");
// Some url we don't want to sync
user_pref("testing.unsynced.url", "moz-extension://d5d31b00-b944-4afb-bd3d-d0326551a0ae");
user_pref(
"testing.unsynced.url",
"moz-extension://d5d31b00-b944-4afb-bd3d-d0326551a0ae"
);
user_pref("testing.synced.url", "https://www.example.com");
// A pref that exists but still has the default value - will be synced with