simplify state logic according to react docs to reduce bugs #88
davemarcoAnnotations
20 errors and 8 warnings
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L1
[import-newlines/enforce] Imports must be broken into multiple lines if there are more than 1 elements.
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L1
[@stylistic/js/object-curly-newline] Expected a line break after this opening brace.
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L1
[@stylistic/js/object-curly-newline] Expected a line break before this closing brace.
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L18
[import-newlines/enforce] Imports must not be broken into multiple lines if there are 1 or less elements.
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L18
[@stylistic/js/object-curly-newline] Unexpected line break after this opening brace.
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L20
[@stylistic/js/object-curly-newline] Unexpected line break before this closing brace.
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L30
[prefer-const] 'numPages' is never reassigned. Use 'const' instead.
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L31
[@stylistic/js/comma-spacing] A space is required after ','.
|
|
new-log-viewer/src/components/MenuBar/PageNumInput.tsx#L11
[import-newlines/enforce] Imports must not be broken into multiple lines if there are 1 or less elements.
|
|
new-log-viewer/src/components/MenuBar/PageNumInput.tsx#L11
[@stylistic/js/object-curly-newline] Unexpected line break after this opening brace.
|
|
new-log-viewer/src/components/MenuBar/PageNumInput.tsx#L13
[@stylistic/js/object-curly-newline] Unexpected line break before this closing brace.
|
|
new-log-viewer/src/components/MenuBar/PageNumInput.tsx#L33
[prefer-const] 'numPages' is never reassigned. Use 'const' instead.
|
|
new-log-viewer/src/components/MenuBar/PageNumInput.tsx#L34
[@stylistic/js/comma-spacing] A space is required after ','.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L185
[prefer-const] 'newLogEventNum' is never reassigned. Use 'const' instead.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L188
[@stylistic/js/semi] Missing semicolon.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L198
[@stylistic/js/semi] Missing semicolon.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L268
[@stylistic/js/indent] Expected indentation of 12 spaces but found 16.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L292
[@stylistic/js/semi] Missing semicolon.
|
|
lint-check
Process completed with exit code 1.
|
|
lint-check
18 errors and 8 warnings
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L1
[simple-import-sort/imports] Run autofix to sort these imports!
|
|
new-log-viewer/src/components/MenuBar/NavigationBar.tsx#L31
[@stylistic/js/max-len] This line has a length of 107. Maximum allowed is 100.
|
|
new-log-viewer/src/components/MenuBar/PageNumInput.tsx#L1
[simple-import-sort/imports] Run autofix to sort these imports!
|
|
new-log-viewer/src/components/MenuBar/PageNumInput.tsx#L34
[@stylistic/js/max-len] This line has a length of 107. Maximum allowed is 100.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L151
[@stylistic/js/max-len] This line has a length of 104. Maximum allowed is 100.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L206
[react-hooks/exhaustive-deps] React Hook useCallback has a missing dependency: 'logEventNum'. Either include it or remove the dependency array.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L271
[react-hooks/exhaustive-deps] React Hook useEffect has a missing dependency: 'pageNum'. Either include it or remove the dependency array.
|
|
new-log-viewer/src/contexts/StateContextProvider.tsx#L295
[react-hooks/exhaustive-deps] React Hook useEffect has a missing dependency: 'logEventNum'. Either include it or remove the dependency array.
|