Skip to content

feat: Implement Step-by-Step Cairo Code Highlighting - Issue #124 #122

feat: Implement Step-by-Step Cairo Code Highlighting - Issue #124

feat: Implement Step-by-Step Cairo Code Highlighting - Issue #124 #122

Triggered via pull request March 27, 2024 05:20
Status Failure
Total duration 1m 21s
Artifacts

ci.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

1 error and 3 warnings
lint
Process completed with exit code 1.
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint: components/Editor/index.tsx#L127
React Hook useEffect has missing dependencies: 'cairoLocation', 'casmToSierraMap', 'decorations', 'monaco', and 'sierraStatements'. Either include them or remove the dependency array
lint: components/Editor/index.tsx#L300
The ref value 'timeoutRef.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'timeoutRef.current' to a variable inside the effect, and use that variable in the cleanup function