Skip to content

Commit d2f4f33

Browse files
authored
Extract type for modeled method kind (#2835)
1 parent 4f1a92d commit d2f4f33

File tree

3 files changed

+9
-7
lines changed

3 files changed

+9
-7
lines changed

extensions/ql-vscode/src/model-editor/modeled-method.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ export interface ModeledMethod extends MethodSignature {
2323
type: ModeledMethodType;
2424
input: string;
2525
output: string;
26-
kind: string;
26+
kind: ModeledMethodKind;
2727
provenance: Provenance;
2828
}
29+
30+
export type ModeledMethodKind = string;

extensions/ql-vscode/src/view/model-editor/KindInput.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
import * as React from "react";
22
import { ChangeEvent, useCallback, useEffect, useMemo } from "react";
3-
import type { ModeledMethod } from "../../model-editor/modeled-method";
3+
import type { ModeledMethodKind } from "../../model-editor/modeled-method";
44
import { Dropdown } from "../common/Dropdown";
55

66
type Props = {
7-
kinds: Array<ModeledMethod["kind"]>;
7+
kinds: ModeledMethodKind[];
88

9-
value: ModeledMethod["kind"] | undefined;
9+
value: ModeledMethodKind | undefined;
1010
disabled?: boolean;
11-
onChange: (value: ModeledMethod["kind"]) => void;
11+
onChange: (value: ModeledMethodKind) => void;
1212

1313
"aria-label"?: string;
1414
};
@@ -29,7 +29,7 @@ export const KindInput = ({
2929
(e: ChangeEvent<HTMLSelectElement>) => {
3030
const target = e.target as HTMLSelectElement;
3131

32-
onChange(target.value as ModeledMethod["kind"]);
32+
onChange(target.value);
3333
},
3434
[onChange],
3535
);

extensions/ql-vscode/src/view/model-editor/MethodRow.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ function ModelableMethodRow(props: MethodRowProps) {
9292

9393
onChange(method, {
9494
...modeledMethod,
95-
output: target.value as ModeledMethod["output"],
95+
output: target.value,
9696
});
9797
},
9898
[onChange, method, modeledMethod],

0 commit comments

Comments
 (0)