Skip to content

Conversation

@romradchenko
Copy link
Owner

@romradchenko romradchenko commented Dec 29, 2025

Every change is important, so it’s better to verify everything when introducing new updates. Accuracy matters, which is why I also run tests in the examples and make sure everything is consistent for users. It runs very fast, so there’s no need to split this into stages for now.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds lefthook for automated Git hook management, specifically setting up pre-commit checks that run linting, typechecking, and tests before commits. The changes also consolidate and clean up .gitignore patterns across the repository.

  • Added lefthook dependency and configured it to run comprehensive checks on pre-commit
  • Created npm scripts for build, typecheck, test, and a unified check command
  • Centralized .gitignore patterns by moving common patterns to the root and simplifying example-specific ignore files

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
package.json Added lefthook dependency, created npm scripts for build/test/typecheck workflows, and added prepare script for automatic lefthook installation
package-lock.json Added lefthook dependency with platform-specific binaries for cross-platform support
lefthook.yml Created lefthook configuration with pre-commit hook that runs the check command
.gitignore Added .next-admin directory pattern to centralize ignore rules
examples/next-admin-auth/.gitignore Simplified by removing patterns now covered by root .gitignore (test-results, playwright-report, .next-admin)
examples/vite-react-auth/.gitignore Created new file to ignore .auth and .env files specific to this example

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@romradchenko romradchenko merged commit 0b89a83 into main Dec 29, 2025
6 checks passed
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