Skip to content
/ utils Public

Change the upload-git-commit-notion language#512

Merged
5ouma merged 2 commits intomainfrom
chore-pre-commit-upload-git-commit-notion-language
Nov 17, 2025
Merged

Change the upload-git-commit-notion language#512
5ouma merged 2 commits intomainfrom
chore-pre-commit-upload-git-commit-notion-language

Conversation

@5ouma
Copy link
Owner

@5ouma 5ouma commented Nov 17, 2025

The script language is deprecated and should be migrated.

@5ouma
chore(pre-commit): Change the upload-git-commit-notion language
8227105 
The `script` language is depricated and should migrate.
@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 •
edited
Loading

Walkthrough

The .pre-commit-hooks.yaml file was modified to update the language field of the upload-git-commit-notion hook from script to unsupported_script. This is a single value replacement affecting how the pre-commit framework processes this hook configuration.

Changes

Cohort / File(s) Summary
Pre-commit hook configuration
.pre-commit-hooks.yaml		 Updated language field value from script to unsupported_script for upload-git-commit-notion hook

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: updating the language field for the upload-git-commit-notion hook in pre-commit configuration.
Description check ✅ Passed The description explains the rationale for the change by noting that the script language is deprecated and needs migration.
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-language
📜 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 9902068 and 0bc3853.

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

5-5: Code change is correct—unsupported_script is the official pre-commit alias for the deprecated script language

The unsupported_script alias was introduced in pre-commit v4.4.0 as the replacement for the deprecated language: script, as documented in the official pre-commit documentation and release notes. No changes are required.

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.

Repository owner deleted a comment from coderabbitai bot Nov 17, 2025
@5ouma
Copy link
Owner Author

5ouma commented Nov 17, 2025

@coderabbitai review

@coderabbitai
Copy link

coderabbitai bot commented Nov 17, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@5ouma 5ouma merged commit 9a342b4 into main Nov 17, 2025
4 of 5 checks passed
@5ouma 5ouma deleted the chore-pre-commit-upload-git-commit-notion-language branch November 17, 2025 14:36
@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