Skip to content

Add Prettier, lint fixes, and typecheck scripts#29

Merged
rickstaa merged 3 commits intomainfrom
chore/add-formatter
Apr 3, 2026
Merged

Add Prettier, lint fixes, and typecheck scripts#29
rickstaa merged 3 commits intomainfrom
chore/add-formatter

Conversation

@rickstaa
Copy link
Copy Markdown
Member

@rickstaa rickstaa commented Apr 3, 2026

Summary

  • Add Prettier, typecheck, and formatting npm scripts
  • Fix all ESLint warnings to pass --max-warnings 0 (unused vars, missing alt, ternary-as-statement)
  • Disable no-img-element globally (project uses raw <img> by design)
  • Remove redundant per-file eslint-disable comments
  • Migrate from deprecated next lint to ESLint CLI

Stacked on #28

Test plan

  • npm run lint passes with --max-warnings 0
  • npm run build compiles successfully
  • Verify no visual regressions from removed unused components

🤖 Generated with Claude Code

rickstaa and others added 3 commits April 3, 2026 12:46
Add prettier and eslint-config-prettier for consistent code formatting.
New scripts: typecheck, format, format:check. Update lint to use
--max-warnings 0 for stricter enforcement.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Remove unused variables (ROWS, ready, children, i, h)
- Fix ternary expression used as statement in LiveNetwork
- Add alt attr to opengraph img
- Disable no-img-element globally (project uses raw <img> by design)
- Remove now-redundant eslint-disable comments

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace `next lint` with `eslint . --max-warnings 0` to avoid the
Next.js 16 deprecation warning. Add ignores for .next/, node_modules/,
and next-env.d.ts in the flat config.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 2026

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

Project Deployment Actions Updated (UTC)
website Ready Ready Preview, Comment Apr 3, 2026 10:56am

Request Review

@rickstaa
Copy link
Copy Markdown
Member Author

rickstaa commented Apr 3, 2026

@adamsoffer please check if the lint fixes make sense or you want to change the rules instead. The blur logic we could reintroduce if you want.

@rickstaa rickstaa mentioned this pull request Apr 3, 2026
3 tasks
Base automatically changed from feat/add-dev-container to main April 3, 2026 16:26
@rickstaa rickstaa merged commit 18efb35 into main Apr 3, 2026
2 of 3 checks passed
@rickstaa rickstaa deleted the chore/add-formatter branch April 3, 2026 16:27
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