Bug 1863400 - Part 8: Update stylelint, eslint, prettierignore, hgignore, gitignore and Generated.txt rules for aboutwelcome to match newtab. r=pdahiya

This also re-runs npm install on browser/components/aboutwelcome to make sure everything in
package-lock.json is up-to-date.

Depends on D193122

Differential Revision: https://phabricator.services.mozilla.com/D193228
This commit is contained in:
Mike Conley 2023-11-28 01:37:38 +00:00
parent 8f2ec8be56
commit 37abcde317
9 changed files with 6507 additions and 3863 deletions

View file

@ -232,6 +232,7 @@ module.exports = {
"browser/base/content/**",
"browser/components/Browser*",
"browser/components/aboutlogins/**",
"browser/components/aboutwelcome/**",
"browser/components/attribution/**",
"browser/components/customizableui/**",
"browser/components/downloads/**",

3
.gitignore vendored
View file

@ -59,6 +59,9 @@ security/manager/.nss.checkout
# Ignore newtab component build assets
browser/components/newtab/logs/
# Ignore about:welcome component build assets
browser/components/aboutwelcome/logs/
# Ignore ASRouter generated test files
browser/components/newtab/content-src/asrouter/schemas/corpus/CFRMessageProvider.messages.json
browser/components/newtab/content-src/asrouter/schemas/corpus/OnboardingMessageProvider.messages.json

View file

@ -57,6 +57,9 @@ compile_commands\.json
# Ignore newtab component build assets
^browser/components/newtab/logs/
# Ignore about:welcome build assets
^browser/components/aboutwelcome/logs/
# Ignore ASRouter generated test files
^browser/components/newtab/content-src/asrouter/schemas/corpus/CFRMessageProvider.messages.json
^browser/components/newtab/content-src/asrouter/schemas/corpus/OnboardingMessageProvider.messages.json
@ -193,6 +196,7 @@ _OPT\.OBJ/
^tools/browsertime/node_modules/
^tools/lint/eslint/eslint-plugin-mozilla/node_modules/
^browser/components/newtab/node_modules/
^browser/components/aboutwelcome/node_modules/
^tools/esmify/node_modules/
# Ignore talos virtualenv and tp5n files.

View file

@ -1032,6 +1032,9 @@ browser/components/pocket/content/panels/js/main.bundle.js
browser/components/newtab/data/
browser/components/newtab/logs/
# Ignore about:welcome files
browser/components/aboutwelcome/logs/
# The only file in browser/locales/ is pre-processed.
browser/locales/
# Generated data files
@ -1443,6 +1446,7 @@ xpcom/io/crc32c.c
##############################################################################
.gradle/
browser/components/aboutwelcome/content/aboutwelcome.bundle.js
browser/components/newtab/content-src/asrouter/schemas/BackgroundTaskMessagingExperiment.schema.json
browser/components/newtab/content-src/asrouter/schemas/MessagingExperiment.schema.json
browser/components/newtab/logs/

View file

@ -25,6 +25,7 @@ obj*/
# These files are generated in some way.
browser/components/pocket/content/panels/css/main.compiled.css
browser/components/newtab/**/*.css
browser/components/aboutwelcome/**/*.css
# Note that the debugger has its own stylelint setup, but that currently
# produces errors. Bug 1831302 tracks making this better

View file

@ -19,7 +19,7 @@ module.exports = {
overrides: [
{
// Only mark the files as modules which are actually modules.
files: ["content-src/**"],
files: ["content-src/**", "tests/unit/**"],
parserOptions: {
sourceType: "module",
},
@ -42,7 +42,7 @@ module.exports = {
},
},
{
files: ["./*.js", "content-src/**"],
files: ["./*.js", "content-src/**", "tests/unit/**"],
env: {
node: true,
},
@ -50,13 +50,13 @@ module.exports = {
{
// Use a configuration that's appropriate for modules, workers and
// non-production files.
files: ["tests/**"],
files: ["modules/*.jsm", "tests/**"],
rules: {
"no-implicit-globals": "off",
},
},
{
files: ["content-src/**"],
files: ["content-src/**", "tests/unit/**"],
rules: {
// Disallow commonjs in these directories.
"import/no-commonjs": 2,
@ -64,6 +64,19 @@ module.exports = {
"react/jsx-no-bind": 0,
},
},
{
// These tests simulate the browser environment.
files: "tests/unit/**",
env: {
browser: true,
mocha: true,
},
globals: {
assert: true,
chai: true,
sinon: true,
},
},
{
files: "tests/**",
rules: {

File diff suppressed because it is too large Load diff

File diff suppressed because it is too large Load diff

View file

@ -1,4 +1,7 @@
.gradle/
browser/components/aboutwelcome/content/aboutwelcome.bundle.js
browser/components/aboutwelcome/logs/
browser/components/aboutwelcome/node_modules/
browser/components/newtab/content-src/asrouter/schemas/BackgroundTaskMessagingExperiment.schema.json
browser/components/newtab/content-src/asrouter/schemas/MessagingExperiment.schema.json
browser/components/newtab/logs/