docs: outputs: gelf: general doc updates and cleanup#2433
docs: outputs: gelf: general doc updates and cleanup#2433eschabell merged 1 commit intofluent:masterfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughDocumentation update to the GELF configuration guide: configuration table keys renamed from mixed case (e.g., Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@pipeline/outputs/gelf.md`:
- Line 19: The table entry for `gelf_timestamp_key` contains a sentence
fragment; update the description to be a complete sentence (for example: "Your
log timestamp. This should be set in GELF.") so the second clause has a subject
and reads as a full sentence; edit the `gelf_timestamp_key` row in
pipeline/outputs/gelf.md to replace "Should be set in GELF." with a complete
sentence like the suggested phrasing.
- Lowercase all config parameter Key column entries - Fix incorrect default listings in config parameter table - Sort configuration parameters table alphabetically - Fix config parameter table column alignment - Remove trailing spaces on gelf_full_message_key row - Fix missing space in mode description: "Allowed values:`tls`" -> "Allowed values: `tls`" - Fix `gelf_timestamp_key` description: "Should be set in GELF." → "This should be set in GELF." Applies to fluent#2412 Signed-off-by: Eric D. Schabell <eric@schabell.org>
8fbf75b to
8f33219
Compare
|
@cosmo0920 and @patrick-stephens ready for review! |
tls" -> "Allowed values:tls"Applies to #2412
Summary by CodeRabbit
Documentation