Linting changes - #73
Open
Levetonus wants to merge 12 commits into
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Levetonus
marked this pull request as ready for review
August 8, 2026 22:22
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
========
ESLint Fixes
According to Codex:
Unescaped JSX apostrophes/quotes in about, FAQ, event, and sponsorship text.
Fixed by replacing raw quotes with JSX-safe entities like ' and ".
Unused imports and variables across nav, sponsors, team, title, event, game, last-year, photos, schedule, and prizes pages.
Fixed by removing dead imports, unused constants, placeholder state, and unused helper code.
any usage in collage, sponsor cards, face cards, sponsorship mouse logic, and sponsorship page helpers.
Fixed by adding focused prop/event types and using unknown for caught errors.
React hook rule violations from synchronous state updates inside effects.
Fixed by moving browser-width initialization into useState initializers and simplifying client-only state in home/nav/title/buffer components.
React purity violation from Math.random() during render in mobile parallax clouds.
Fixed by replacing render-time randomness with deterministic seeded values.
React immutability violation from mutating a Three.js texture returned by a hook.
Fixed by removing the direct mutation.
Invalid top-level hook usage in the mobile Three worker.
Fixed by removing useState from module scope and rendering the scene directly.
Lowercase React page component name in prizes.
Fixed by renaming prizes to Prizes.
arguments usage in prizes.
Fixed by removing the unused local animation helper that used it.
CommonJS require("daisyui") in Tailwind config.
Fixed by switching to an ES import and adding types/daisyui.d.ts.
TypeScript checking build output under out.
Fixed by excluding out in tsconfig.json.
Project lint policy warnings for plain
and one-time GSAP/Lenis effects.
Fixed in eslint.config.mjs by allowing plain image tags for this visual-heavy site and disabling exhaustive deps for intentionally one-time animation setup effects.