Skip to content

[nightshift] lint-fix: automated lint fixes#2

Open
nightshift-micr wants to merge 1 commit intomicr-dev:mainfrom
nightshift-micr:nightshift/lint-fix
Open

[nightshift] lint-fix: automated lint fixes#2
nightshift-micr wants to merge 1 commit intomicr-dev:mainfrom
nightshift-micr:nightshift/lint-fix

Conversation

@nightshift-micr
Copy link
Copy Markdown
Contributor

Automated by Nightshift v3 (GLM 5.1).

Task: lint-fix
Category: pr
Changes: Fixed 5 ESLint errors in components/skiper/skiper80.tsx:

  1. react-hooks/refs (2 occurrences): Replaced render-time loadedImagesRef.current read with a loadedImages state variable (Set<string>), keeping the ref for callback-side checks.
  2. react-hooks/set-state-in-effect (3 occurrences): Wrapped synchronous state setters in requestAnimationFrame to avoid cascading renders during useEffect execution.

ESLint passes clean (0 errors), TypeScript compiles without errors.

Merge if useful, close if not.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 17, 2026

@nightshift-micr is attempting to deploy a commit to the Projects Team on Vercel.

A member of the Team first needs to authorize it.

@Microck Microck added dayshift/pr-inbox Managed by hermes-dayshift-glm dayshift/in-progress Managed by hermes-dayshift-glm dayshift/failed Managed by hermes-dayshift-glm and removed dayshift/pr-inbox Managed by hermes-dayshift-glm dayshift/failed Managed by hermes-dayshift-glm dayshift/in-progress Managed by hermes-dayshift-glm labels Apr 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants