0

Disable misc custom eslint rules.

Details:
* Removing components/.eslintrc.js had no effect.
* content/browser/resources/.eslintrc.js was hoisted into toplevel
  .eslintrc.js
* tools/typescript/definitions/.eslintrc.js was unnecessary since
  linting is unnecessary for auto-generated files.

Bug: 369766161, 368085620
Change-Id: I43586614ddc9ed9f6deb12e435c0be4d567f8dbf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5896508
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1362150}
This commit is contained in:
Erik Chen
2024-10-01 02:04:53 +00:00
committed by Chromium LUCI CQ
parent c674df7f55
commit ce7c10aa1b
4 changed files with 5 additions and 38 deletions
.eslintrc.js
components
content/browser/resources
tools/typescript/definitions

@@ -300,7 +300,8 @@ module.exports = {
// We do not allow per-directory custom eslint rules. This section exists for
// rules that are in the process of being applied to the whole code base.
{
'files': ['ui/webui/resources/**/*.[jt]s'],
'files': ['content/browser/resources/**/*.[jt]s',
'ui/webui/resources/**/*.[jt]s',],
'rules': {
'eqeqeq': ['error', 'always', {'null': 'ignore'}],
}
@@ -319,6 +320,9 @@ module.exports = {
'chrome/test/data/webui/chromeos/cr_focus_row_behavior_interactive_test.js',
'chrome/test/data/webui/chromeos/**/*_browsertest.js',
// No point linting auto-generated files.
'tools/typescript/definitions/**',
// ESLint is disabled for camera_app_ui and recorder_app_ui as they
// used a custom eslint plugin that does not work with the latest eslint,
// and they had complex eslint rc files that have not been updated to the

@@ -1,9 +0,0 @@
// Copyright 2019 The Chromium Authors
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
module.exports = {
'rules' : {
'no-restricted-properties' : 'off',
},
};

@@ -1,13 +0,0 @@
// Copyright 2022 The Chromium Authors
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
module.exports = {
'env': {
'browser': true,
'es6': true,
},
'rules': {
'eqeqeq': ['error', 'always', {'null': 'ignore'}],
},
};

@@ -1,15 +0,0 @@
// Copyright 2024 The Chromium Authors
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
module.exports = {
'rules' : {
// Turn off since files in this folder are autogenerated, and the
// tool does not always generate names in line with naming convention.
'@typescript-eslint/naming-convention': 'off',
// Turn off since files in this folder are autogenerated and do not
// exist at runtime.
'@typescript-eslint/consistent-type-imports': 'off',
},
};