Skip to content

out_loki: fix 'occured' -> 'occurred' in comment#11725

Merged
edsiper merged 1 commit intofluent:masterfrom
SAY-5:fix-loki-occured-typo
Apr 16, 2026
Merged

out_loki: fix 'occured' -> 'occurred' in comment#11725
edsiper merged 1 commit intofluent:masterfrom
SAY-5:fix-loki-occured-typo

Conversation

@SAY-5
Copy link
Copy Markdown
Contributor

@SAY-5 SAY-5 commented Apr 16, 2026

Comment in plugins/out_loki/loki.c line 2004 reads Server-side error occured. Fixed to occurred. Comment-only change.

Summary by CodeRabbit

  • Chores
    • Corrected spelling in internal documentation.

Comment in plugins/out_loki/loki.c line 2004 read 'Server-side error occured'. Fixed to 'occurred'. Comment-only change.

Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 77a8fd26-d002-4728-a7d0-13726a0d680f

📥 Commits

Reviewing files that changed from the base of the PR and between b6859e3 and 0758fd2.

📒 Files selected for processing (1)
  • plugins/out_loki/loki.c

📝 Walkthrough

Walkthrough

A spelling correction in a comment within the Loki output plugin, changing "occured" to "occurred" in the cb_loki_flush function description of server-side error handling. No functional or behavioral changes.

Changes

Cohort / File(s) Summary
Comment Spelling Correction
plugins/out_loki/loki.c
Fixed spelling of "occurred" in a comment describing error conditions in the cb_loki_flush function.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A typo hopped right into view,
"Occurred" fixed in comment true,
No logic changed, just words made right,
The Loki plugin shines more bright! 🐰✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: a spelling correction from 'occured' to 'occurred' in a comment in the out_loki plugin file.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@edsiper edsiper merged commit 856d762 into fluent:master Apr 16, 2026
3 checks passed
@edsiper
Copy link
Copy Markdown
Member

edsiper commented Apr 16, 2026

thank you

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants