Skip to content

fix(lib): Remove default pre-commit hook to prevent commit b#23

Merged
JFusco merged 1 commit intomainfrom
feat/det-install
Apr 2, 2026
Merged

fix(lib): Remove default pre-commit hook to prevent commit b#23
JFusco merged 1 commit intomainfrom
feat/det-install

Conversation

@JFusco
Copy link
Copy Markdown
Collaborator

@JFusco JFusco commented Apr 2, 2026

fix(lib): Remove default pre-commit hook to prevent commit b

Summary (AI, bounded)

  • Remove the default pre-commit hook to prevent commit blocking, with related updates in lib/init-workspace.js, bin/cli.js, and README.md (230923e)

Changes

  • 230923e fix(lib): Remove default pre-commit hook to prevent commit b

Files changed

  • M README.md
  • M bin/cli.js
  • M lib/init-workspace.js

Diff stats

README.md             |  4 ++-
 bin/cli.js            |  8 +++++
 lib/init-workspace.js | 87 +++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 98 insertions(+), 1 deletion(-)

Testing

  • Manual smoke test
  • CI green

Risk / Impact

  • Risk level: Low / Medium / High
  • Rollback plan:

Base: main → Head: feat/det-install

Suggested labels / Review checklist

Suggested labels: fix, lib, testing, docs

Review checklist:

  • Verify removal of pre-commit hook does not break commit workflow
  • Confirm README.md updates accurately reflect changes
  • Run manual smoke test to ensure CLI and workspace initialization work as expected
  • Ensure CI passes without errors related to these changes

This change removes the default pre-commit hook created by Husky, which
typically runs tests using npm, pnpm, or yarn. Such a hook can block
commits if tests fail or take too long to execute. By eliminating this
default hook, users are encouraged to create their own pre-commit hooks
or rely on continuous integration for testing.

The removal of the default pre-commit hook ensures a smoother
development experience, allowing developers to commit changes without
being hindered by failing tests. This change promotes flexibility in how
developers manage their commit processes.

No issue references detected.
@JFusco JFusco marked this pull request as ready for review April 2, 2026 13:48
@JFusco JFusco merged commit 6bc5e74 into main Apr 2, 2026
3 checks passed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 2, 2026

🎉 This PR is included in version 2.5.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@JFusco JFusco deleted the feat/det-install branch April 2, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant