Skip to content

Commit 7d4317a

Browse files
committed
Enable more ESLint rules
1 parent 1aa407b commit 7d4317a

29 files changed

+70
-31
lines changed

eslint.config.mjs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,12 @@ export default [
5252
"@typescript-eslint/switch-exhaustiveness-check": "error",
5353
"@typescript-eslint/consistent-type-definitions": "off",
5454
"@typescript-eslint/no-namespace": ["error", { allowDeclarations: true }],
55+
"@typescript-eslint/consistent-type-exports": "error",
56+
"@typescript-eslint/consistent-type-imports": [
57+
"error",
58+
{ fixStyle: "inline-type-imports" },
59+
],
60+
"@typescript-eslint/no-import-type-side-effects": "error",
5561
},
5662
},
5763
];

src/api/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as v from "@badrap/valita";
2-
import { HTTPError, Client, ClientConfig } from "./client.js";
2+
import { HTTPError, Client, type ClientConfig } from "./client.js";
33
import { Kv } from "./kv.js";
44

55
export { HTTPError };

src/api/kv.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as v from "@badrap/valita";
2-
import { Client, HTTPError } from "./client.js";
2+
import { type Client, HTTPError } from "./client.js";
33

44
type KvKey = (number | string | boolean)[];
55

src/jsx-dev-runtime.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1-
import { jsx, Fragment, JSX } from "./jsx-runtime.js";
1+
import { jsx, Fragment, type JSX } from "./jsx-runtime.js";
22

3-
export { jsx as jsxDEV, Fragment, JSX };
3+
export type { JSX };
4+
export { jsx as jsxDEV, Fragment };

src/jsx-runtime.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,13 @@
11
import {
2-
Json,
3-
UiNode,
4-
FunctionalComponent,
2+
type Json,
3+
type UiNode,
4+
type FunctionalComponent,
55
Fragment,
66
element,
77
} from "./ui/internal.js";
88

9-
export { UiNode, FunctionalComponent, Fragment };
9+
export type { UiNode, FunctionalComponent };
10+
export { Fragment };
1011

1112
type ClassList = string | Record<string, boolean> | ClassList[];
1213

src/ui/experimental/Alert.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { UiNode, element } from "../internal.js";
1+
import { type UiNode, element } from "../internal.js";
22

33
type AlertProps = {
44
/**

src/ui/experimental/Box.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,9 @@
1-
import { Responsive, Space, UiNode, element } from "../internal.js";
1+
import {
2+
type Responsive,
3+
type Space,
4+
type UiNode,
5+
element,
6+
} from "../internal.js";
27

38
type BoxProps = {
49
width?: Responsive<string>;

src/ui/experimental/Button.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,9 @@
1-
import { Json, UiNode, element, Responsive } from "../internal.js";
1+
import {
2+
type Json,
3+
type UiNode,
4+
element,
5+
type Responsive,
6+
} from "../internal.js";
27

38
type ButtonProps = {
49
variant?: "default" | "primary" | "danger";

src/ui/experimental/Card.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { UiNode, element } from "../internal.js";
1+
import { type UiNode, element } from "../internal.js";
22

33
type CardProps = {
44
/**

src/ui/experimental/Checkbox.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { Json, UiNode, element } from "../internal.js";
1+
import { type Json, type UiNode, element } from "../internal.js";
22

33
type CheckboxProps = {
44
name?: string;

0 commit comments

Comments
 (0)