[chore] Lint and formatting changes (#7599)

ggdouglas · web-flow · commit 8aec14bfe7f3 · 2025-10-24T10:24:30.000-04:00
diff --git a/packages/colors/src/index.ts b/packages/colors/src/index.ts
@@ -15,4 +15,5 @@
  */
 
 export { Colors } from "./colors";
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export { LegacyColors } from "./legacyColors";
diff --git a/packages/core/karma.conf.js b/packages/core/karma.conf.js
@@ -19,8 +19,8 @@ module.exports = async function (config) {
                 "src/accessibility/*",
                 "src/common/abstractComponent*",
                 "src/common/abstractPureComponent*",
-                "src/common/alignment.ts",      
-                "src/common/buttonVariant.ts",     
+                "src/common/alignment.ts",
+                "src/common/buttonVariant.ts",
                 "src/common/size.ts",
                 "src/common/errors.ts",
                 "src/components/html/html.tsx",
diff --git a/packages/core/src/common/index.ts b/packages/core/src/common/index.ts
@@ -23,6 +23,7 @@ export { Boundary } from "./boundary";
 export { ButtonVariant } from "./buttonVariant";
 export { Elevation } from "./elevation";
 export { Intent } from "./intent";
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export { KeyCodes as Keys } from "./keyCodes";
 export { Position } from "./position";
 export {
diff --git a/packages/core/src/components/hotkeys/index.ts b/packages/core/src/components/hotkeys/index.ts
@@ -23,7 +23,10 @@ export {
     HotkeysTarget,
     type HotkeysTargetProps,
     type HotkeysTargetRenderProps,
+    // eslint-disable-next-line @typescript-eslint/no-deprecated
     HotkeysTarget2,
+    // eslint-disable-next-line @typescript-eslint/no-deprecated
     type HotkeysTarget2Props,
+    // eslint-disable-next-line @typescript-eslint/no-deprecated
     type HotkeysTarget2RenderProps,
 } from "./hotkeysTarget";
diff --git a/packages/core/src/components/index.ts b/packages/core/src/components/index.ts
@@ -70,11 +70,13 @@ export { NavbarGroup, type NavbarGroupProps } from "./navbar/navbarGroup";
 export { NavbarHeading, type NavbarHeadingProps } from "./navbar/navbarHeading";
 export { NonIdealState, type NonIdealStateProps, NonIdealStateIconSize } from "./non-ideal-state/nonIdealState";
 export { OverflowList, type OverflowListProps } from "./overflow-list/overflowList";
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export { Overlay } from "./overlay/overlay";
 export type { OverlayLifecycleProps, OverlayProps, OverlayableProps } from "./overlay/overlayProps";
 export { Overlay2, type Overlay2Props } from "./overlay2/overlay2";
 export type { OverlayInstance } from "./overlay2/overlayInstance";
 export { Text, type TextProps } from "./text/text";
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export { PanelStack, type PanelStackProps, PanelStack2, type PanelStack2Props } from "./panel-stack/panelStack";
 export type { Panel, PanelProps } from "./panel-stack/panelTypes";
 export { Popover } from "./popover/popover";
@@ -121,6 +123,7 @@ export { Tag, type TagProps } from "./tag/tag";
 export { TagInput, type TagInputProps, type TagInputAddMethod } from "./tag-input/tagInput";
 export { OverlayToaster, type OverlayToasterCreateOptions } from "./toast/overlayToaster";
 export type { OverlayToasterProps, ToasterPosition } from "./toast/overlayToasterProps";
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export { Toast, Toast2 } from "./toast/toast";
 export type { ToastProps } from "./toast/toastProps";
 export type { Toaster, ToastOptions } from "./toast/toaster";
diff --git a/packages/eslint-plugin/test/no-deprecated-components.test.ts b/packages/eslint-plugin/test/no-deprecated-components.test.ts
@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-/* eslint-disable no-template-curly-in-string, sort-keys */
-
 import { RuleTester } from "@typescript-eslint/rule-tester";
 import dedent from "dedent";
 
diff --git a/packages/eslint-plugin/test/no-deprecated-core-components.test.ts b/packages/eslint-plugin/test/no-deprecated-core-components.test.ts
@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-/* eslint-disable no-template-curly-in-string, sort-keys */
-
 import { RuleTester } from "@typescript-eslint/rule-tester";
 import dedent from "dedent";
 
diff --git a/packages/eslint-plugin/test/no-deprecated-popover2-components.test.ts b/packages/eslint-plugin/test/no-deprecated-popover2-components.test.ts
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-/* eslint-disable no-template-curly-in-string, sort-keys */
+/* eslint-disable sort-keys */
 
 import { RuleTester } from "@typescript-eslint/rule-tester";
 import dedent from "dedent";
diff --git a/packages/eslint-plugin/test/no-deprecated-select-components.test.ts b/packages/eslint-plugin/test/no-deprecated-select-components.test.ts
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-/* eslint-disable no-template-curly-in-string, sort-keys */
+/* eslint-disable sort-keys */
 
 import { RuleTester } from "@typescript-eslint/rule-tester";
 import dedent from "dedent";
diff --git a/packages/eslint-plugin/test/no-deprecated-table-components.test.ts b/packages/eslint-plugin/test/no-deprecated-table-components.test.ts
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-/* eslint-disable no-template-curly-in-string, sort-keys */
+/* eslint-disable sort-keys */
 
 import { RuleTester } from "@typescript-eslint/rule-tester";
 import dedent from "dedent";
diff --git a/packages/eslint-plugin/test/no-deprecated-type-references.test.ts b/packages/eslint-plugin/test/no-deprecated-type-references.test.ts
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-/* eslint-disable no-template-curly-in-string, sort-keys */
+/* eslint-disable sort-keys */
 
 import { RuleTester } from "@typescript-eslint/rule-tester";
 import dedent from "dedent";
diff --git a/packages/landing-app/src/logo.ts b/packages/landing-app/src/logo.ts
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-/* eslint-disable max-classes-per-file, no-cond-assign */
+/* eslint-disable max-classes-per-file */
 
 const LOGO_Y_OFFSET = 250;
 const SHADOW_DEPTH = 0.3;
@@ -111,7 +111,6 @@ export class Matrix {
 
 const M = (m?: MatrixTuple) => new Matrix(m);
 
-// eslint-disable-line no-shadow
 export abstract class Transformable<T extends Transformable<any>> {
     public abstract transform(matrix: Matrix): T;
 
@@ -184,7 +183,6 @@ export class Quaternion {
     }
 
     public toMatrix() {
-        // eslint-disable-line @typescript-eslint/no-this-alias
         const { w, x, y, z } = this;
         return M([
             1 - 2 * (y * y + z * z),
diff --git a/packages/table/src/cell/editableCell.tsx b/packages/table/src/cell/editableCell.tsx
@@ -268,6 +268,7 @@ export class EditableCell extends Component<EditableCellProps, EditableCellState
 
 /** @deprecated Use `EditableCell` instead */
 export const EditableCell2 = EditableCell;
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export type EditableCell2 = InstanceType<typeof EditableCell2>;
 
 /** @deprecated Use `EditableCellProps` instead */
diff --git a/packages/table/src/index.ts b/packages/table/src/index.ts
@@ -16,6 +16,7 @@
 
 export { Cell, type CellProps, type CellRenderer } from "./cell/cell";
 
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export { EditableCell, EditableCell2, type EditableCellProps, type EditableCell2Props } from "./cell/editableCell";
 
 export { JSONFormat, type JSONFormatProps } from "./cell/formats/jsonFormat";
@@ -71,6 +72,8 @@ export {
     TableLoadingOption,
 } from "./regions";
 
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export type { TableProps, Table2Props } from "./tableProps";
 
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export { Table, Table2 } from "./table";
diff --git a/packages/table/src/table.tsx b/packages/table/src/table.tsx
@@ -1632,4 +1632,5 @@ export class Table extends AbstractComponent<TableProps, TableState, TableSnapsh
 
 /** @deprecated Use `Table` instead */
 export const Table2 = Table;
+// eslint-disable-next-line @typescript-eslint/no-deprecated
 export type Table2 = InstanceType<typeof Table2>;
diff --git a/packages/test-commons/src/utils.ts b/packages/test-commons/src/utils.ts
@@ -93,7 +93,7 @@ export function dispatchTouchEvent(target: EventTarget, eventType = "touchstart"
  * We can't simply call mount() here since React 16 throws before we can even validate the errors thrown
  * in component constructors.
  */
-// eslint-disable-next-line @typescript-eslint/no-empty-object-type
+
 export function expectPropValidationError<P extends object>(
     Component: React.ComponentClass<P>,
     props: P & { children?: React.ReactNode },
@@ -102,7 +102,7 @@ export function expectPropValidationError<P extends object>(
 ) {
     const { defaultProps = {} } = Component;
     // HACKHACK: weird casts ahead
-    // eslint-disable-next-line @typescript-eslint/no-empty-object-type
+
     expect(() => new Component({ ...(defaultProps as object), ...(props as object) } as P)).to.throw(
         errorMessage,
         assertionMessage,
