Skip to content

Commit 25d9059

Browse files
committed
fix: restructure to use only 1 assertion
1 parent bc57a97 commit 25d9059

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

src/deferred-promise.ts

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,11 @@ export class DeferredPromise<T> extends Promise<T> {
1313
executor: (resolve: (value: T) => void, reject: (reason: Error) => void) => void,
1414
{ timeout, onTimeout }: DeferredPromiseOptions<T> = {}
1515
) {
16-
let resolveFn: (value: T) => void;
17-
let rejectFn: (reason?: unknown) => void;
18-
1916
super((resolve, reject) => {
20-
resolveFn = resolve;
21-
rejectFn = reject;
17+
this.resolve = resolve;
18+
this.reject = reject;
2219
});
2320

24-
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
25-
this.resolve = resolveFn!;
26-
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
27-
this.reject = rejectFn!;
28-
2921
if (timeout !== undefined && onTimeout) {
3022
this.timeoutId = setTimeout(() => {
3123
onTimeout(this.resolve, this.reject);

0 commit comments

Comments
 (0)