Skip to content

Conversation

@llstrk
Copy link
Contributor

@llstrk llstrk commented Oct 2, 2025

Updated links for roadmap, build-and-install and binaries. The IBM Z page in the documentation does not seem to have existed.


Enter [N/A] in the box, if an item is not applicable to your change.

Testing
Before we can approve your change; please submit the following in a comment:

  • [N/A] Example configuration file for the change
  • [N/A] Debug log output from testing the change
  • [N/A] Attached Valgrind output that shows no leaks or memory corruption was found

If this is a change to packaging of containers or native binaries then please confirm it works for all targets.

  • [N/A] Run local packaging test showing all targets (including any new ones) build.
  • [N/A] Set ok-package-test label to test for all targets (requires maintainer to do).

Documentation

  • [N/A] Documentation required for this feature

Backporting

  • [N/A] Backport to latest stable release.

Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

Summary by CodeRabbit

  • Documentation
    • Updated README links to new official URLs.
      • Roadmap → /Fluent-Bit-Roadmap
      • Build & Install → downloads/source/build-and-install
      • Install links updated:
        • Linux → downloads/linux
        • Docker → downloads/docker
        • Windows → downloads/windows
      • Removed IBM Z (s390x) link
    • No functional code changes.

Updated links for roadmap, build-and-install and binaries. The IBM Z page in the documentation does not seem to have existed.

Signed-off-by: Dennis Rye <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Oct 2, 2025

Walkthrough

README.md was updated to revise external links: roadmap path changed, Build & Install now points to downloads/source/build-and-install, Install Fluent Bit links redirected to downloads/linux, downloads/docker, and downloads/windows, and the IBM Z (s390x) link was removed. No code or API changes.

Changes

Cohort / File(s) Summary
Docs: README link updates
README.md
Replaced roadmap URL (/Roadmap → /Fluent-Bit-Roadmap); updated Build & Install link to downloads/source/build-and-install; adjusted Install Fluent Bit links to downloads/linux, downloads/docker, downloads/windows; removed IBM Z (s390x) link.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I nudge the links, hop-hop, precise,
From old trails to fresh, concise.
Linux, Docker, Windows—pathways bright,
Roadmap gleams in cleaner light.
No code to chew, just docs to trim—
A tidy warren, neat and prim. 🐇✨

Pre-merge checks and finishing touches

✅ 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 “docs: fix broken links” succinctly captures the primary change of updating documentation URLs to correct broken references, directly reflecting the content of the changeset and providing clear context for reviewers and future maintainers.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aea8830 and 98c3858.

📒 Files selected for processing (1)
  • README.md (3 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-09-14T09:46:09.531Z
Learnt from: aminvakil
PR: fluent/fluent-bit#10844
File: conf/fluent-bit:13-15
Timestamp: 2025-09-14T09:46:09.531Z
Learning: For fluent-bit Debian packaging, /opt/fluent-bit/bin/ is the appropriate installation path since the package may be installed from non-official Debian sources, making /opt compliant with FHS for optional software packages.

Applied to files:

  • README.md
🔇 Additional comments (3)
README.md (3)

33-33: Roadmap link update looks correct.

The wiki path aligns with the current roadmap location.


69-69: Build & Install reference confirmed.

The new URL matches the downloads/source/build-and-install page.


81-83: Install links point to canonical locations.

Linux, Docker, and Windows URLs now resolve under the downloads section.


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.

Copy link
Collaborator

@patrick-stephens patrick-stephens left a comment

Choose a reason for hiding this comment

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

Thanks for that, I think the docs got refactored recently which moved some of these links around so appreciate you resolving them.

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