'setTheme' is never reassigned. Use 'const' instead
4import useLocalStorage from "use-local-storage";
5
6export default function ThemeToggle() {
7 let [theme, setTheme] = useTheme(); 8 const [themeStorage, setThemeStorage] = useLocalStorage("theme", "light");
9 let themeBindings = {
10 "light": "light_mode",
'themeBindings' is never reassigned. Use 'const' instead
6export default function ThemeToggle() {
7 let [theme, setTheme] = useTheme();
8 const [themeStorage, setThemeStorage] = useLocalStorage("theme", "light");
9 let themeBindings = {10 "light": "light_mode",
11 "dark": "dark_mode"
12 }
'theme' is never reassigned. Use 'const' instead
4import useLocalStorage from "use-local-storage";
5
6export default function ThemeToggle() {
7 let [theme, setTheme] = useTheme(); 8 const [themeStorage, setThemeStorage] = useLocalStorage("theme", "light");
9 let themeBindings = {
10 "light": "light_mode",
Description
Variables that are never re-assigned a new value after their initial declaration should be declared with the const
keyword.
This prevents the programmer from erroneously re-assigning to a read-only variable, and informs those reading the code that a variable is a constant value.
Bad Practice
let pi = Math.PI
for (let x of xs) {
use(x);
}
let { a, b } = object;
use(a, b);
Recommended
const pi = Math.PI
for (const x of xs) {
use(x);
}
const { a, b } = object;
use(a, b);