Repo: ESLint warnings in useSandboxServices (react-hooks/exhaustive-deps)

[JoshuaKGoldberg]

Suggestion

/home/runner/work/typescript-eslint/typescript-eslint/packages/website/src/components/editor/useSandboxServices.ts
Warning:   128:6  warning  React Hook useEffect has missing dependencies: 'colorMode', 'onLoaded', and 'props'. Either include them or remove the dependency array  react-hooks/exhaustive-deps

Example: #6907 -> https://github.com/typescript-eslint/typescript-eslint/actions/runs/4703925812/jobs/8343076171?pr=6907

This is showing up in PRs. Can we fix for the rule's complaint without causing bugs in the playground?

[armano2]

sadly we have to eslint-disable it

[armano2]

this has been fixed in v6

typescript-eslint/packages/website/src/components/editor/useSandboxServices.ts

Lines 150 to 153 in f1a0d78

	// colorMode and jsx can't be reactive here because we don't want to force a recreation
	// updating of colorMode and jsx is handled in LoadedEditor
	// eslint-disable-next-line react-hooks/exhaustive-deps
	}, [props.ts, onLoaded]);