Skip to content

docs: outputs: dynatrace: fix diverse doc issues#2427

Merged
patrick-stephens merged 1 commit intofluent:masterfrom
eschabell:erics_out_dynatrace_updates
Mar 16, 2026
Merged

docs: outputs: dynatrace: fix diverse doc issues#2427
patrick-stephens merged 1 commit intofluent:masterfrom
eschabell:erics_out_dynatrace_updates

Conversation

@eschabell
Copy link
Collaborator

@eschabell eschabell commented Mar 16, 2026

  • Sort configuration parameters table alphabetically
  • Normalize table first-column padding to match other output plugin docs
  • Wrap header authorization default value in backticks
  • Wrap {your-API-token-here} and {your-environment-id} placeholders in backticks
  • Apply Title_Case consistently to all classic config keys Allow_duplicated_headers, Host, Uri, Json_date_format, Json_date_key, Tls, Tls.verify
  • Fix URI -> Uri and remove inconsistent mixed casing

Applies to #2412

Summary by CodeRabbit

  • Documentation
    • Improved Dynatrace output configuration documentation with standardized table formatting and header alignment.
    • Updated configuration example formatting with consistent key naming and structure for enhanced clarity.

  - Sort configuration parameters table alphabetically
  - Normalize table first-column padding to match other output plugin docs
  - Wrap header authorization default value in backticks
  - Wrap {your-API-token-here} and {your-environment-id} placeholders in backticks
  - Apply Title_Case consistently to all classic config keys
    Allow_duplicated_headers, Host, Uri, Json_date_format, Json_date_key, Tls, Tls.verify
  - Fix URI -> Uri and remove inconsistent mixed casing

  Applies to fluent#2412

Signed-off-by: Eric D. Schabell <eric@schabell.org>
@eschabell eschabell self-assigned this Mar 16, 2026
@eschabell eschabell requested a review from a team as a code owner March 16, 2026 09:35
@eschabell eschabell added waiting-on-review Waiting on a review from mainteners 5.0 labels Mar 16, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 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: 28b182c8-2ec3-4f1a-8fe2-f6ffa177894c

📥 Commits

Reviewing files that changed from the base of the PR and between 69238cc and 05c3ea4.

📒 Files selected for processing (1)
  • pipeline/outputs/dynatrace.md

📝 Walkthrough

Walkthrough

This pull request updates the Dynatrace output configuration documentation by reformatting tables from multi-line to single-line header format, reordering parameter entries, and converting configuration example keys from lowercase to capitalized styling for consistency.

Changes

Cohort / File(s) Summary
Dynatrace Documentation
pipeline/outputs/dynatrace.md
Reformatted configuration reference tables with aligned headers, reordered parameters, and updated example configurations from lowercase to capitalized key formatting (e.g., nameName, matchMatch).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • cosmo0920
  • patrick-stephens

Poem

📝 A documentation dance so fine,
Headers aligned in perfect line,
Lowercase keys now capitalize bright,
Tables reformatted just right,
The Dynatrace guide shines anew! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 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: fixing diverse documentation issues in the Dynatrace output plugin docs, including table reorganization, normalization, and formatting standardization.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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.

@eschabell
Copy link
Collaborator Author

@coderabbitai review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 16, 2026

✅ 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.

@eschabell
Copy link
Collaborator Author

eschabell commented Mar 16, 2026

@cosmo0920 and @patrick-stephens ready for review!

@patrick-stephens patrick-stephens merged commit 7045f1b into fluent:master Mar 16, 2026
8 checks passed
@eschabell eschabell removed the waiting-on-review Waiting on a review from mainteners label Mar 16, 2026
@eschabell eschabell deleted the erics_out_dynatrace_updates branch March 16, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants