Revert "Commit hook: Warn about files with significant changes in next (#4095)"

[olemartinorg]

Reverts #4095

Summary by CodeRabbit

Chores

• Simplified the pre-commit hook configuration by removing monorepo path validation checks and streamlining the commit validation process to run linting directly.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 22cb303b-ee75-4f83-8e60-baa2938e53f8

📥 Commits

Reviewing files that changed from the base of the PR and between 557ab7e and fce6ef8.

📒 Files selected for processing (2)

• .husky/pre-commit
• .husky/pre-commit-check-monorepo-paths

💤 Files with no reviewable changes (2)

• .husky/pre-commit
• .husky/pre-commit-check-monorepo-paths

---

📝 Walkthrough

Walkthrough

The pre-commit Git hook is simplified to remove a monorepo path validation step. The hook no longer invokes a check script that blocked commits when staged files matched a remote list of changed/refactored paths. The associated shell script is deleted.

Changes

Pre-commit hook simplification

Layer / File(s)	Summary
Remove monorepo path validation from pre-commit flow .husky/pre-commit, .husky/pre-commit-check-monorepo-paths	The pre-commit hook removes its invocation of the monorepo path check script. The check script file, which previously fetched remote paths and blocked commits on matches, is deleted entirely.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is incomplete. While it mentions the revert target, it lacks detailed explanation of changes, testing verification, and required checklist items specified in the repository template.	Add a detailed description of what was reverted, mark appropriate testing and verification checkboxes, and add required labels (kind/* and backport* are partially addressed but need clarity).

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: reverting a previous commit hook warning feature, which matches the actual removal of the monorepo path check and the pre-commit hook modification.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch revert-4095-chore/monorepo-commit-hook

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}