Skip to content

Commit

Permalink
fix lint errors
Browse files Browse the repository at this point in the history
  • Loading branch information
yeonjuan committed Apr 1, 2024
1 parent 5d0489f commit 00b1e57
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 10 deletions.
20 changes: 11 additions & 9 deletions packages/website/src/components/editor/LoadedEditor.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -135,14 +135,14 @@ export const LoadedEditor: React.FC<LoadedEditorProps> = ({
monaco.editor.setModelMarkers(model, 'eslint', diagnostics);
updateMarkers();
});
return () => disposable();
return (): void => disposable();
}, [webLinter, monaco, codeActions, updateMarkers]);

useEffect(() => {
const disposable = webLinter.onParse((uri, model) => {
onASTChange(model);
});
return () => disposable();
return (): void => disposable();
}, [webLinter, onASTChange]);

useEffect(() => {
Expand Down Expand Up @@ -178,7 +178,7 @@ export const LoadedEditor: React.FC<LoadedEditorProps> = ({
'typescript',
createProvideCodeActions(codeActions),
);
return () => disposable.dispose();
return (): void => disposable.dispose();
}, [codeActions, monaco]);

useEffect(() => {
Expand All @@ -191,7 +191,9 @@ export const LoadedEditor: React.FC<LoadedEditorProps> = ({
}
}
});
return () => disposable.dispose();
return (): void => {
disposable.dispose();
};
}, [editor, tabs.eslintrc]);

useEffect(() => {
Expand All @@ -204,7 +206,7 @@ export const LoadedEditor: React.FC<LoadedEditorProps> = ({
}
}, 150),
);
return () => disposable.dispose();
return (): void => disposable.dispose();
}, [onSelect, editor, tabs.code]);

useEffect(() => {
Expand All @@ -227,7 +229,7 @@ export const LoadedEditor: React.FC<LoadedEditorProps> = ({
}
},
});
return () => disposable.dispose();
return (): void => disposable.dispose();
}, [editor, monaco, webLinter]);

useEffect(() => {
Expand All @@ -243,7 +245,7 @@ export const LoadedEditor: React.FC<LoadedEditorProps> = ({
}),
];

return () => {
return (): void => {
closable.forEach(c => c.close());
};
}, [system, onChange]);
Expand All @@ -255,14 +257,14 @@ export const LoadedEditor: React.FC<LoadedEditorProps> = ({
system.writeFile(model.uri.path, model.getValue());
}
});
return () => disposable.dispose();
return (): void => disposable.dispose();
}, [editor, system]);

useEffect(() => {
const disposable = monaco.editor.onDidChangeMarkers(() => {
updateMarkers();
});
return () => disposable.dispose();
return (): void => disposable.dispose();
}, [monaco.editor, updateMarkers]);

const resize = useMemo(() => {
Expand Down
2 changes: 1 addition & 1 deletion packages/website/src/hooks/useMediaQuery.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ const useMediaQuery = (mediaQuery: string): boolean => {
}

documentChangeHandler();
return () => {
return (): void => {
try {
mediaQueryList.removeEventListener('change', documentChangeHandler);
} catch {
Expand Down

0 comments on commit 00b1e57

Please sign in to comment.