Skip to content
/ utils Public

Always print upload-git-commit-notion logs#510

Merged
5ouma merged 1 commit intomainfrom
chore-pre-commit-upload-git-commit-notion-verbose
Nov 17, 2025
Merged

Always print upload-git-commit-notion logs#510
5ouma merged 1 commit intomainfrom
chore-pre-commit-upload-git-commit-notion-verbose

Conversation

@5ouma
Copy link
Owner

@5ouma 5ouma commented Nov 17, 2025

It's useful to see how many commits are uploaded.

@5ouma
chore(pre-commit): Always print upload-git-commit-notion logs
cc9d230 
It's useful to see how many commits are uploaded.
@5ouma 5ouma 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-hooks.yaml configuration file is updated to add a verbose: true setting to the upload-git-commit-notion pre-commit hook, enabling verbose output during hook execution. No other hook properties or behaviors are modified.

Changes

Cohort / File(s) Summary
Pre-commit Configuration
\.pre-commit-hooks.yaml``		 Added verbose: true flag to the upload-git-commit-notion hook to enable verbose output during execution

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Minimal single-line configuration addition with no logic or behavioral changes
  • No dependencies or cascading effects on other configurations

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: enabling verbose output for the upload-git-commit-notion pre-commit hook to display logs.
Description check ✅ Passed The description is related to the changeset, explaining the practical benefit of the change by noting it helps see how many commits are uploaded.
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 chore-pre-commit-upload-git-commit-notion-verbose
📜 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 6c7547e and cc9d230.

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

7-7: LGTM! Clean and focused configuration change.

The addition of verbose: true correctly enables verbose output for the hook, which aligns with the PR objective to display upload progress information. The YAML syntax and indentation are consistent with the existing 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 and usage tips.

@5ouma 5ouma merged commit 0cecac5 into main Nov 17, 2025
5 checks passed
@5ouma 5ouma deleted the chore-pre-commit-upload-git-commit-notion-verbose branch November 17, 2025 13:49
@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