chore(tests): fix duplicated "the" in filter_throttle_size comments#11801
chore(tests): fix duplicated "the" in filter_throttle_size comments#11801quyentonndbs wants to merge 1 commit into
Conversation
Signed-off-by: Kai Tanaka <275430420+quyentonndbs@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates comment text in three test verification sections of the throttle size filter test file without modifying any executable code or test logic. ChangesTest Comment Wording
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Ah, we need to use a specific commit prefix like as:
[2](https://github.com/fluent/fluent-bit/actions/runs/25846360090/job/75967346555?pr=11801#step:6:13)
❌ Commit df0a736b54 failed:
Missing prefix in commit subject: 'chore(tests): fix duplicated "the" in filter_throttle_size comments'
Commit prefix validation failed.
Removes a doubled
the thefrom three comment lines intests/runtime/filter_throttle_size.c. Comment-only, no test behavior change.Summary by CodeRabbit