Skip to content

Linting changes - #73

Open
Levetonus wants to merge 12 commits into
mainfrom
linting-changes
Open

Linting changes#73
Levetonus wants to merge 12 commits into
mainfrom
linting-changes

Conversation

@Levetonus

@Levetonus Levetonus commented Aug 7, 2026

Copy link
Copy Markdown
Contributor
  • Replaced react-multi-carousel with embla-carousel-react and rewrote the last year page to remove vulnerabilities
  • Added ESLint (and the hundreds of errors were fixed)
  • Hydration issues were fixed by adding useSyncExternalStore in nav-bar.tsx and title.tsx
  • Added Prettier and reformatted the code
  • Relevant section of CONTRIBUTING.md has been updated
  • Removed unused packages and updated existing ones

========

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.

@vercel

vercel Bot commented Aug 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
super-secret-hackrpi Ready Ready Preview Sep 4, 2026 10:23pm UTC

@Levetonus
Levetonus marked this pull request as ready for review August 8, 2026 22:22
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.

1 participant