Skip to content

chore: Update imperatives.py to add auto#417

Merged
shenxianpeng merged 1 commit into
mainfrom
shenxianpeng-patch-1
May 26, 2026
Merged

chore: Update imperatives.py to add auto#417
shenxianpeng merged 1 commit into
mainfrom
shenxianpeng-patch-1

Conversation

@shenxianpeng
Copy link
Copy Markdown
Member

@shenxianpeng shenxianpeng commented May 26, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Chores
    • Enhanced commit message validation to restrict additional imperative verbs.

Review Change Stack

@shenxianpeng shenxianpeng requested a review from a team as a code owner May 26, 2026 06:02
@netlify
Copy link
Copy Markdown

netlify Bot commented May 26, 2026 •
edited
Loading

✅ Deploy Preview for commit-check ready!

Name Link
🔨 Latest commit 7fb54df
🔍 Latest deploy log https://app.netlify.com/projects/commit-check/deploys/6a15375b82973500085b2c56
😎 Deploy Preview https://deploy-preview-417--commit-check.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 26, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@shenxianpeng shenxianpeng merged commit f5525d9 into main May 26, 2026
13 of 16 checks passed
@shenxianpeng shenxianpeng deleted the shenxianpeng-patch-1 branch May 26, 2026 06:03
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 26, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f9523640-f8a8-41b3-9050-8cb05465f890

📥 Commits

Reviewing files that changed from the base of the PR and between ad53628 and 7fb54df.

📒 Files selected for processing (1)
  • commit_check/imperatives.py
📝 Walkthrough

Walkthrough

The IMPERATIVES verb blacklist in commit_check/imperatives.py is extended by adding the "auto" imperative string to the set, which will now be flagged as disallowed in commit messages.

Changes

Imperative blacklist update

Layer / File(s) Summary
Add 'auto' imperative verb
commit_check/imperatives.py		 The IMPERATIVES set is updated to include the imperative "auto", extending the verb blacklist for commit message validation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

enhancement

Poem

🐰 Auto, auto, what a word,
Now blacklisted and roundly heard!
The imperatives grow, the checks grow keen,
Cleanest commits you've ever seen! ✨

✨ 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 shenxianpeng-patch-1

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.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 26, 2026

Merging this PR will not alter performance

✅ 302 untouched benchmarks
⏩ 108 skipped benchmarks1

Comparing shenxianpeng-patch-1 (7fb54df) with main (e44b79a)2

Open in CodSpeed

Footnotes

  1. 108 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

  2. No successful run was found on main (ad53628) during the generation of this report, so e44b79a was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@shenxianpeng