ci: added react-you-might-not-need-an-effect lint #236
Annotations
11 warnings
|
Upload test results
No files were found with the provided path: playwright-report/. No artifacts will be uploaded.
|
|
Lint:
src/components/ThemeToggle.tsx#L13
Avoid initializing state in an effect. Instead, pass "mounted"'s initial value to its `useState`
|
|
Lint:
src/components/TaskManager.tsx#L74
Avoid using state and effects as an event handler. Instead, call the event handling code directly when the event occurs
|
|
Lint:
src/components/TaskManager.tsx#L64
Avoid using state and effects as an event handler. Instead, call the event handling code directly when the event occurs
|
|
Lint:
src/components/TabbedCodeBlock.tsx#L53
Avoid chaining state changes. When possible, update all relevant state simultaneously
|
|
Lint:
src/components/TabbedCodeBlock.tsx#L44
Avoid chaining state changes. When possible, update all relevant state simultaneously
|
|
Lint:
src/components/Conversations.tsx#L230
Avoid using state and effects as an event handler. Instead, call the event handling code directly when the event occurs
|
|
Lint:
src/components/Conversations.tsx#L218
Avoid using state and effects as an event handler. Instead, call the event handling code directly when the event occurs
|
|
Lint:
src/components/ChatInput.tsx#L292
Avoid passing data to parents in an effect. Instead, let the parent fetch the data itself and pass it down to the child as a prop
|
|
Lint:
src/components/ChatInput.tsx#L251
Avoid storing derived state. Compute "selectedModel" directly during render, optionally with `useMemo` if it's expensive
|
|
Lint:
src/components/BrowserPreview.tsx#L29
Avoid chaining state changes. When possible, update all relevant state simultaneously
|
Loading