Skip to content

Chore/add pre commit script#15

Merged
Nasrallah-AL merged 5 commits intoFoothillSolutions:mainfrom
engkareeem:chore/add-pre-commit-script
Apr 20, 2026
Merged

Chore/add pre commit script#15
Nasrallah-AL merged 5 commits intoFoothillSolutions:mainfrom
engkareeem:chore/add-pre-commit-script

Conversation

@engkareeem
Copy link
Copy Markdown
Contributor

This pull request introduces a pre-commit hook using Husky to enforce code quality checks before allowing commits, and updates the project dependencies accordingly. It also removes two unused dependencies related to Hocuspocus.

Dependency and tooling updates:

  • Added husky as a development dependency and a prepare script to set up Git hooks, enabling automated checks before each commit. [1] [2]
  • Removed @hocuspocus/provider and @hocuspocus/server from the dependencies, because they are redudant.

Pre-commit hook setup:

  • Added a .husky/pre-commit script that stashes unstaged changes, runs type checking, linting, and tests via bun run check, and blocks the commit if any checks fail. The script also ensures unstaged changes are restored after checks.

- Added husky to manage pre-commit hooks.
- Created a pre-commit hook to run checks before commits.
- Updated package.json to include husky as a dependency.
- Added git stash commands to save and restore changes during pre-commit checks.
- Ensured that the exit code from the checks is preserved for proper error handling.
- Added a condition to skip checks if no changes are staged.
- Enhanced user feedback during the pre-commit process with clear messages for stashing, running checks, and restoring changes.
- Ensured that the commit is blocked if any checks fail, providing better error handling.est
- Introduced a cleanup function to restore unstaged changes after pre-commit checks.
- Added a flag to track whether changes were stashed, enhancing the reliability of the process.
- Improved messaging for better user feedback during the pre-commit workflow.
- Added a message to indicate the completion of checks in the pre-commit hook.
- Improved user feedback during the pre-commit process for better clarity.
@Nasrallah-AL Nasrallah-AL merged commit 9e66a4b into FoothillSolutions:main Apr 20, 2026
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