Skip to content

chore: remove husky and local commit hooks - #23

Merged
fengzai6 merged 2 commits into
mainfrom
chore/remove-husky
Jul 14, 2026
Merged

chore: remove husky and local commit hooks#23
fengzai6 merged 2 commits into
mainfrom
chore/remove-husky

Conversation

@fengzai6

@fengzai6 fengzai6 commented Jul 14, 2026

Copy link
Copy Markdown
Owner

💻 变更类型 | Change Type

  • feat
  • fix
  • refactor
  • chore
  • style
  • docs
  • test

🔀 变更说明 | Description of Change

📝 补充信息 | Additional Information

🔗 关联 Issue | Related Issue

Summary by CodeRabbit

  • 变更
    • 移除本地提交信息校验与提交前检查。
    • 提交规范改由 CI 和代码审查进行约束。
    • 清理相关配置与开发依赖,简化项目配置。

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
my-first-nest Ready Ready Preview, Comment Jul 14, 2026 12:27am

@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

移除 Husky、lint-staged 和 commitlint 本地校验配置、依赖及钩子脚本,调整服务构建排除项,并将 AGENTS.md 与 README.md 中的提交规范更新为由 CI 和 Code Review 约束。

Changes

本地提交校验移除

Layer / File(s) Summary
移除提交校验工具
package.json, apps/server/tsconfig.build.json
删除 Husky 初始化脚本、commitlint 与相关开发依赖,并从构建排除项中移除 commitlint.config.ts
同步开发规范说明
AGENTS.md, README.md
技术栈和提交规范说明不再要求本地 Husky/commitlint 校验,改为由 CI 与 Code Review 约束。

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • fengzai6/my-first-nest#9:同样涉及移除 commitlint.config.ts、Husky 提交校验及 TypeScript 构建排除配置。
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题准确概括了本次移除 Husky 和本地提交钩子的主要变更。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-husky

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@AGENTS.md`:
- Around line 132-133: Update the commit-message policy wording in AGENTS.md
lines 132-133 and README.md line 340 to match the actual workflow: remove the
claim that CI enforces commit conventions and state that compliance relies on
Code Review and repository conventions; do not add commitlint or CI checks.
🪄 Autofix (Beta)

✅ Autofix completed


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 47f36008-fae4-48e0-8cdb-3216e1359506

📥 Commits

Reviewing files that changed from the base of the PR and between 2c5efc3 and 79cf898.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (7)
  • .husky/commit-msg
  • .husky/pre-commit
  • AGENTS.md
  • README.md
  • apps/server/tsconfig.build.json
  • commitlint.config.ts
  • package.json
💤 Files with no reviewable changes (4)
  • .husky/commit-msg
  • commitlint.config.ts
  • apps/server/tsconfig.build.json
  • .husky/pre-commit

Comment thread AGENTS.md Outdated
@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Fixes Applied Successfully

Fixed 2 file(s) based on 1 unresolved review comment.

Files modified:

  • AGENTS.md
  • README.md

Commit: 8a46a162c3999b749163a8d4029d6843f269c8ae

The changes have been pushed to the chore/remove-husky branch.

Time taken: 2m 46s

Fixed 2 file(s) based on 1 unresolved review comment.

Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
@fengzai6
fengzai6 merged commit 81265d4 into main Jul 14, 2026
4 checks passed
@fengzai6
fengzai6 deleted the chore/remove-husky branch July 14, 2026 00:29
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