Skip to content

chore(repo): CI, pre-commit hooks, lint cleanup, and dev guard#2

Closed
jal-co wants to merge 2 commits intomainfrom
feat/npm-badge
Closed

chore(repo): CI, pre-commit hooks, lint cleanup, and dev guard#2
jal-co wants to merge 2 commits intomainfrom
feat/npm-badge

Conversation

@jal-co
Copy link
Owner

@jal-co jal-co commented Mar 13, 2026

Summary

Post-merge follow-ups from the npm-badge PR plus repo quality infrastructure.

Changes

  • CI workflow — GitHub Actions runs lint, type check, registry build, and full build on every PR
  • Pre-commit hooks — Husky + lint-staged: ESLint on staged files + full type check before commit
  • Lint cleanup — Migrate from deprecated next lint to eslint ., ignore .next/ generated files, fix unused vars in json-viewer and tip-jar
  • Dev guard/dev/screenshots returns 404 in production via notFound()
  • Screenshots — Add npm-badge dark/light preview PNGs

Checklist

  • pnpm lint passes with zero errors
  • pnpm build passes
  • Pre-commit hook tested and working

jal-co added 2 commits March 12, 2026 19:53
- GitHub Actions CI: lint, type check, registry build, full build on PRs
- Husky pre-commit: lint staged files + type check
- Add husky and lint-staged dev dependencies
- Migrate lint script from deprecated next lint to eslint CLI
- Ignore .next/ and next-env.d.ts generated files
- Remove unused CopyButton in json-viewer
- Remove unused isCrypto in tip-jar tabs variant
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