build: remove linting from docs site

Removes the linting from the docs site directory so that we can consolidate it with the rest of the project.

Author: crisbeto

.github/workflows/ci.material-aio.yml

@@ -17,20 +17,6 @@ defaults:
     shell: bash
 
 jobs:
-  lint:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Initialize environment
-        uses: angular/dev-infra/github-actions/npm/checkout-and-setup-node@e4bf37af223483ce00f9316d227fd62cd744dc4b
-      - name: Setup Bazel
-        uses: angular/dev-infra/github-actions/bazel/setup@e4bf37af223483ce00f9316d227fd62cd744dc4b
-      - name: Setup Bazel RBE
-        uses: angular/dev-infra/github-actions/bazel/configure-remote@e4bf37af223483ce00f9316d227fd62cd744dc4b
-      - name: Install node modules
-        run: pnpm install --frozen-lockfile
-      - name: Execute Linting
-        run: pnpm bazel test --test_tag_filters=lint //docs/...
-
   build:
     runs-on: ubuntu-latest
     steps:
@@ -57,7 +43,7 @@ jobs:
       - name: Install node modules
         run: pnpm install --frozen-lockfile
       - name: Execute Tests
-        run: pnpm bazel test --test_tag_filters=-lint,-e2e,-audit //docs/...
+        run: pnpm bazel test --test_tag_filters=-e2e,-audit //docs/...
       - name: Store Test Logs
         uses: actions/upload-artifact@834a144ee995460fba8ed112a2fc961b36a5ec5a # v4.3.6
         if: always()

.github/workflows/pr.material-aio.yml

@@ -15,20 +15,6 @@ defaults:
     shell: bash
 
 jobs:
-  lint:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Initialize environment
-        uses: angular/dev-infra/github-actions/npm/checkout-and-setup-node@e4bf37af223483ce00f9316d227fd62cd744dc4b
-      - name: Setup Bazel
-        uses: angular/dev-infra/github-actions/bazel/setup@e4bf37af223483ce00f9316d227fd62cd744dc4b
-      - name: Setup Bazel RBE
-        uses: angular/dev-infra/github-actions/bazel/configure-remote@e4bf37af223483ce00f9316d227fd62cd744dc4b
-      - name: Install node modules
-        run: pnpm install --frozen-lockfile
-      - name: Execute Linting
-        run: pnpm bazel test --test_tag_filters=lint //docs/...
-
   build:
     runs-on: ubuntu-latest
     steps:
@@ -55,7 +41,7 @@ jobs:
       - name: Install node modules
         run: pnpm install --frozen-lockfile
       - name: Execute Tests
-        run: pnpm bazel test --test_tag_filters=-lint,-e2e,-audit //docs/...
+        run: pnpm bazel test --test_tag_filters=-e2e,-audit //docs/...
       - name: Store Test Logs
         uses: actions/upload-artifact@834a144ee995460fba8ed112a2fc961b36a5ec5a # v4.3.6
         if: always()

docs/.eslintrc.json

@@ -26,14 +26,6 @@ copy_to_bin(
     ],
 )
 
-# Lint config files
-copy_to_bin(
-    name = "ng-base-lint-config",
-    srcs = [
-        ".eslintrc.json",
-    ],
-)
-
 # The main application
 ng_app(
     name = "app",

docs/BUILD.bazel

@@ -179,12 +179,6 @@
             ]
           }
         },
-        "lint": {
-          "builder": "@angular-eslint/builder:lint",
-          "options": {
-            "lintFilePatterns": ["src/**/*.ts", "src/**/*.html"]
-          }
-        },
         "e2e": {
           "builder": "@angular-devkit/build-angular:protractor",
           "options": {
@@ -290,12 +284,6 @@
             }
           }
         },
-        "lint": {
-          "builder": "@angular-eslint/builder:lint",
-          "options": {
-            "lintFilePatterns": ["scenes/**/*.ts", "scenes/**/*.html"]
-          }
-        },
         "e2e": {
           "builder": "@angular-devkit/build-angular:protractor",
           "options": {

docs/angular.json

@@ -86,28 +86,6 @@ E2E_DEPS = [
     "//docs:node_modules/webdriver-manager",
 ]
 
-LINT_CONFIG = COMMON_CONFIG + [
-    # Lint uses the e2e config
-    "//docs:ng-base-test-config",
-    ":ng-e2e-config",
-    "//docs:ng-base-lint-config",
-    "//docs:node_modules/@angular-eslint/builder",
-    "//docs:node_modules/@angular-eslint/eslint-plugin",
-    "//docs:node_modules/@angular-eslint/eslint-plugin-template",
-    "//docs:node_modules/@angular-eslint/template-parser",
-    "//docs:node_modules/eslint-plugin-ban",
-    "//docs:node_modules/eslint-plugin-import",
-    "//docs:node_modules/eslint-plugin-jsdoc",
-    "//docs:node_modules/eslint-plugin-prefer-arrow",
-    "//docs:node_modules/@typescript-eslint/eslint-plugin",
-    "//docs:node_modules/@typescript-eslint/parser",
-    "//docs:node_modules/@stylistic/eslint-plugin",
-]
-LINT_DEPS = [
-    # TODO(bazel): this should be included as a transitive of @angular-devkit/architect-cli!?
-    "//docs:node_modules/@angular-devkit/architect",
-]
-
 # buildifier: disable=unused-variable
 def ng_app(name, project_name = None, deps = [], test_deps = [], e2e_deps = [], **kwargs):
     """
@@ -164,16 +142,6 @@ def ng_app(name, project_name = None, deps = [], test_deps = [], e2e_deps = [],
         visibility = ["//visibility:private"],
     )
 
-    # Lint config files in addition to the root
-    if native.package_name() != "":
-        copy_to_bin(
-            name = "lint-config",
-            srcs = [
-                ".eslintrc.json",
-            ],
-            visibility = ["//visibility:private"],
-        )
-
     project_name = project_name if project_name else name
 
     native.alias(
@@ -234,14 +202,6 @@ def ng_app(name, project_name = None, deps = [], test_deps = [], e2e_deps = [],
     #     **kwargs
     # )
 
-    _architect_test(
-        project_name,
-        "lint",
-        srcs = srcs + test_srcs + e2e_srcs + deps + test_deps + NG_COMMON_DEPS + LINT_DEPS + LINT_CONFIG + ([":lint-config"] if native.package_name() != "" else []),
-        tags = tags + ["lint"],
-        **kwargs
-    )
-
 def _architect_build(project_name, configuration = None, args = [], srcs = [], **kwargs):
     output_dir = "%s%s" % (project_name, ".%s" % configuration if configuration else "")
 

docs/defs.bzl

@@ -10,7 +10,6 @@
     "start:prod": "ng serve --configuration production",
     "start:scenes": "ng serve scenes",
     "start:emulators": "firebase emulators:start",
-    "lint": "ng lint",
     "test": "ng test",
     "e2e": "ng e2e",
     "build": "ng build",
@@ -58,24 +57,12 @@
     "@angular-devkit/architect": "catalog:",
     "@angular-devkit/architect-cli": "catalog:",
     "@angular-devkit/build-angular": "catalog:",
-    "@angular-eslint/builder": "19.3.0",
-    "@angular-eslint/eslint-plugin": "19.3.0",
-    "@angular-eslint/eslint-plugin-template": "19.3.0",
-    "@angular-eslint/template-parser": "19.3.0",
     "@angular/cli": "catalog:",
     "@angular/compiler-cli": "catalog:",
     "@bazel/bazelisk": "^1.12.1",
-    "@stylistic/eslint-plugin": "^2.6.2",
     "@types/jasmine": "5.1.7",
     "@types/node": "^22.14.1",
     "@types/shelljs": "0.8.15",
-    "@typescript-eslint/eslint-plugin": "8.29.1",
-    "@typescript-eslint/parser": "8.29.1",
-    "eslint": "^8.56.0",
-    "eslint-plugin-ban": "^1.5.2",
-    "eslint-plugin-import": "^2.22.1",
-    "eslint-plugin-jsdoc": "^48.2.1",
-    "eslint-plugin-prefer-arrow": "^1.2.3",
     "firebase-tools": "^11.0.1",
     "jasmine-core": "5.6.0",
     "jasmine-spec-reporter": "7.0.0",

docs/package.json

@@ -220,7 +220,6 @@ class DocViewerTestComponent {
 }
 
 const FAKE_DOCS: {[key: string]: string} = {
-  /* eslint-disable @typescript-eslint/naming-convention */
   'http://material.angular.io/simple-doc.html': '<div>my docs page</div>',
   'http://material.angular.io/doc-with-example.html': `
       <div>Check out this example:</div>
@@ -259,7 +258,6 @@ const FAKE_DOCS: {[key: string]: string} = {
         data-docs-api-module-import-button="import {MatIconModule} from '@angular/material/icon';">
       </div>
     </div>`,
-  /* eslint-enable @typescript-eslint/naming-convention */
 };
 
 @Component({

docs/scenes/.eslintrc.json

@@ -8,10 +8,8 @@ const testExampleId = 'my-test-example-id';
 const testExampleBasePath = `/docs-content/examples-source/cdk/my-comp/${testExampleId}`;
 
 const FAKE_DOCS: {[key: string]: string} = {
-  /* eslint-disable @typescript-eslint/naming-convention */
   '/assets/stack-blitz/src/index.html': '<material-docs-example></material-docs-example>',
   '/assets/stack-blitz/src/main.ts': `import {MaterialDocsExample} from './material-docs-example';`,
-  /* eslint-enable @typescript-eslint/naming-convention */
   [`${testExampleBasePath}/test.ts`]: 'ExampleComponent',
   [`${testExampleBasePath}/test.html`]: `<example></example>`,
   [`${testExampleBasePath}/src/detail.ts`]: 'DetailComponent',
@@ -102,7 +100,6 @@ describe('StackBlitzWriter', () => {
     flushMicrotasks();
 
     const expectedFiles = jasmine.objectContaining({
-      /* eslint-disable @typescript-eslint/naming-convention */
       'angular.json': 'fake',
       'src/main.ts': `import {ExampleComponent} from './test';`,
       'src/test.ts': 'fake',
@@ -112,7 +109,6 @@ describe('StackBlitzWriter', () => {
 /**  Copyright ${new Date().getFullYear()} Google LLC. All Rights Reserved.
     Use of this source code is governed by an MIT-style license that
     can be found in the LICENSE file at https://angular.io/license */`,
-      /* eslint-enable @typescript-eslint/naming-convention */
     });
 
     expect(openProjectSpy).toHaveBeenCalledTimes(1);

docs/src/app/shared/doc-viewer/doc-viewer.spec.ts

@@ -1,13 +1,11 @@
 import '@angular/localize/init';
-import {importProvidersFrom} from '@angular/core';
 import {bootstrapApplication} from '@angular/platform-browser';
 import {provideHttpClient} from '@angular/common/http';
 import {provideAnimations} from '@angular/platform-browser/animations';
 import {VERSION as CDK_VERSION} from '@angular/cdk';
 import {VERSION as MAT_VERSION, provideNativeDateAdapter} from '@angular/material/core';
 import {MaterialDocsExample} from './example/material-docs-example';
 
-/* eslint-disable no-console */
 console.info('Angular CDK version', CDK_VERSION.full);
 console.info('Angular Material version', MAT_VERSION.full);