Merge pull request #142 from microsoft/users/rbitting/ts-eslint-config

Set up eslint configuration for TypeScript files

Author: aubreyquinn

.eslintrc.js

@@ -25,6 +25,26 @@ module.exports = {
         {
             files: ["tests/**/*.js"],
             env: { mocha: true }
+        },
+        {
+            files: ["tests/**/*.ts"],
+            env: { jest: true }
+        },
+        {
+            files: ["**/*.ts"],
+            parser: "@typescript-eslint/parser",
+            plugins: ["@typescript-eslint"],
+            parserOptions: {
+                ecmaVersion: 2021,
+                sourceType: "module",
+                project: "./tsconfig.json"
+            },
+            rules: {
+                // Disable Node.js rules that conflict with TypeScript
+                "node/no-missing-import": "off",
+                "node/no-unsupported-features/es-syntax": "off",
+                "node/no-extraneous-import": "off"
+            }
         }
     ],
     rules: {

lib/applicableComponents/labelBasedComponents.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
 const labelBasedComponents = ["Label", "label"];
 const elementsUsedAsLabels = ["div", "span", "p", "h1", "h2", "h3", "h4", "h5", "h6"];
 

lib/rules/field-needs-labelling.ts

@@ -3,7 +3,7 @@
 
 "use strict";
 
-const { hasNonEmptyProp } = require("../util/hasNonEmptyProp");
+import { hasNonEmptyProp } from "../util/hasNonEmptyProp";
 const elementType = require("jsx-ast-utils").elementType;
 import { ESLintUtils, TSESTree } from "@typescript-eslint/utils";
 

lib/rules/index.ts

@@ -1,3 +1,6 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
 export { default as accordionHeaderNeedsLabelling } from "./accordion-header-needs-labelling";
 export { default as accordionItemNeedsHeaderAndPanel } from "./accordion-item-needs-header-and-panel";
 export { default as avatarNeedsName } from "./avatar-needs-name";

tests/lib/rules/utils/hasFieldParent.test.ts

@@ -25,9 +25,11 @@ const createMockContext = (ancestors: TSESTree.Node[]): TSESLint.RuleContext<str
     report: jest.fn(),
     getAncestors: () => ancestors,
     getSourceCode: jest.fn(),
+    // eslint-disable-next-line no-unused-vars
     getDeclaredVariables: function (node: TSESTree.Node): readonly TSESLint.Scope.Variable[] {
         throw new Error("Function not implemented.");
     },
+    // eslint-disable-next-line no-unused-vars
     markVariableAsUsed: function (name: string): boolean {
         throw new Error("Function not implemented.");
     }

tests/lib/rules/utils/hasLabelledChilImage.test.ts

@@ -1,4 +1,6 @@
-// Import necessary dependencies and mock functions
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
 import {
     hasLabelledChildImage,
     isImageHidden,

tsconfig.json

@@ -19,6 +19,6 @@
         "rootDir": ".",
         "allowJs": true
     },
-    "include": ["lib"],
+    "include": ["lib", "tests/lib"],
     "exclude": ["node_modules", "dist"]
 }