Skip to content

Updates for Fluent Bit for Developers doc section#2402

Merged
eschabell merged 3 commits intofluent:masterfrom
eschabell:erics_developement_updates
Mar 6, 2026
Merged

Updates for Fluent Bit for Developers doc section#2402
eschabell merged 3 commits intofluent:masterfrom
eschabell:erics_developement_updates

Conversation

@eschabell
Copy link
Collaborator

@eschabell eschabell commented Mar 1, 2026

  • wasm-input-plugins: update Rust target to wasm32-wasip1
  • golang-output-plugins: fix invalid YAML in config example
  • developer-guide: update plugin.c source link to master

Fixes #2401

Summary by CodeRabbit

  • Documentation
    • Improved formatting and readability of plugin API documentation
    • Updated plugin configuration syntax examples for clarity
    • Updated WebAssembly build target references for Rust toolchain compatibility

…ster

Applies to fluent#2401

Signed-off-by: Eric D. Schabell <eric@schabell.org>
…example

Applies to fluent#2401

Signed-off-by: Eric D. Schabell <eric@schabell.org>
…asip1

Applies to fluent#2401

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

coderabbitai bot commented Mar 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dc38f3c and 65c0088.

📒 Files selected for processing (3)
  • development/developer-guide.md
  • development/golang-output-plugins.md
  • development/wasm-input-plugins.md

📝 Walkthrough

Walkthrough

The PR updates the Fluent Bit developers documentation section, fixing inline code formatting and plugin reference links, adjusting plugin configuration syntax in golang-output-plugins documentation, and updating Rust target triples from wasm32-unknown-unknown/wasm32-wasi to wasm32-wasip1 in wasm-input-plugins documentation.

Changes

Cohort / File(s) Summary
Link and Formatting Fixes
development/developer-guide.md
Updated plugin-loading reference links to master branch and corrected backtick formatting around dlopen and dlsym code tokens.
Configuration Syntax Updates
development/golang-output-plugins.md
Changed plugins configuration from list entry to dedicated plugins_file key and updated pipeline inputs section to align with YAML mapping syntax.
Rust Target Triple Updates
development/wasm-input-plugins.md
Updated Rust target triples from wasm32-unknown-unknown and wasm32-wasi to wasm32-wasip1, including corresponding rustup installation commands and documentation references.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Suggested reviewers

  • cosmo0920
  • patrick-stephens

Poem

🐰 A hop, a skip, through docs we bound,
With links corrected and syntax sound,
From wasi old to wasip1 new,
The developer guide is fresh and true! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main objective: updating documentation in the Fluent Bit developer guide section, directly related to the three doc file changes.
Linked Issues check ✅ Passed The PR addresses issue #2401 by updating developer documentation across three files (developer-guide.md, golang-output-plugins.md, wasm-input-plugins.md) to refresh and correct identified issues.
Out of Scope Changes check ✅ Passed All changes are within scope: documentation updates to developer-facing sections (plugin guides, developer guide) directly supporting the refresh objective of issue #2401.
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

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

@cosmo0920 and @patrick-stephens ready for review!

Copy link
Contributor

@cosmo0920 cosmo0920 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, thanks for updating outdated WASM related documents! 👍

@eschabell eschabell merged commit 1481ad4 into fluent:master Mar 6, 2026
8 checks passed
@eschabell eschabell deleted the erics_developement_updates branch March 6, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5.0 waiting-on-review Waiting on a review from mainteners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refresh neeeded for Fluent Bit for Developers section docs

2 participants