Skip to content
/ utils Public

Create a scripts directory for hooks#507

Merged
5ouma merged 1 commit intomainfrom
style-pre-commit-script-path
Nov 17, 2025
Merged

Create a scripts directory for hooks#507
5ouma merged 1 commit intomainfrom
style-pre-commit-script-path

Conversation

@5ouma
Copy link
Owner

@5ouma 5ouma commented Nov 17, 2025

Organize the scripts that will be increased in the future.

@5ouma
style(pre-commit): Create a scripts directory for hooks
b11e201 
Organize the scripts which will be increased in the future.
@github-actions github-actions bot added the 🚸 pre-commit Check the code with pre-commit hooks label Nov 17, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 17, 2025

Walkthrough

The pre-commit hook configuration file has been updated to relocate the entry path for the upload-git-commit-notion hook from pre-commit/upload-git-commit-notion to pre-commit/scripts/upload-git-commit-notion, adding a /scripts subdirectory segment to the script location.

Changes

Cohort / File(s) Change Summary
Pre-commit hook path update
.pre-commit-hooks.yaml		 Updated entry field path from pre-commit/upload-git-commit-notion to pre-commit/scripts/upload-git-commit-notion to reflect new script directory structure

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: creating a scripts directory and reorganizing hook scripts into it.
Description check ✅ Passed The description is related to the changeset, explaining the purpose of organizing scripts for future growth.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch style-pre-commit-script-path
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e6454c7 and b11e201.

📒 Files selected for processing (1)
  • .pre-commit-hooks.yaml (1 hunks)
🔇 Additional comments (1)
.pre-commit-hooks.yaml (1)

4-4: Entry path change verified and complete.

The script has been successfully moved to pre-commit/scripts/upload-git-commit-notion and the configuration has been updated to match. All verification checks confirm the migration is clean: the new path exists, the old path is empty, and there are no orphaned references to the old path in the codebase. The README reference is to the hook ID (which remains unchanged), not the script path, so no documentation updates are needed.

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 and usage tips.

@5ouma 5ouma merged commit 47289ba into main Nov 17, 2025
5 checks passed
@5ouma 5ouma deleted the style-pre-commit-script-path branch November 17, 2025 07:20
@5ouma 5ouma bot mentioned this pull request Nov 17, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

@5ouma 5ouma

Labels

🚸 pre-commit Check the code with pre-commit hooks

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@5ouma