Skip to content

Conversation

@esmerel
Copy link

@esmerel esmerel commented Nov 4, 2025

Updated links for Windows checksums and container versions in documentation.


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:

  • Example configuration file for the change
  • Debug log output from testing the change
  • 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.

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

Documentation

  • Documentation required for this feature

Backporting

  • 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 documentation links to direct users to the downloads section for Windows checksums and container versions.

Updated links for Windows checksums and container versions in documentation.

Signed-off-by: Lynette  Miles <6818907+esmerel@users.noreply.github.com>
@coderabbitai
Copy link

coderabbitai bot commented Nov 4, 2025

Walkthrough

Two documentation links in the GitHub workflows README were updated to direct users to downloads sections instead of installation pages. Windows checksums and container version links now point to their respective download resources. No functional or code changes were made.

Changes

Cohort / File(s) Summary
Documentation link updates
\.github/workflows/README\.md
Updated Windows checksums link to target downloads/windows packages and container versions link to target downloads/docker tags

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • docs: fix broken links #10965: Similar non-functional documentation link updates redirecting installation/download references to new downloads URL sections across multiple docs files.

Suggested reviewers

  • edsiper
  • fujimotos
  • koleini

Poem

🐰 Links hop to new homes,
Windows and containers find their way,
Downloads guide the seekers true,
Documentation springs ahead,
All paths now lead where they should. 📍

Pre-merge checks and finishing touches

✅ 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 'Fix docs links for Windows checksums and container versions' directly and specifically describes the main change: updating documentation links for Windows checksums and container versions.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • 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 ed06c31 and d07b2ac.

📒 Files selected for processing (1)
  • .github/workflows/README.md (1 hunks)
🔇 Additional comments (1)
.github/workflows/README.md (1)

177-178: Documentation links verified as accessible.

Both URLs return HTTP 200 responses and are properly accessible:

  • Windows checksums link ✓
  • Container versions link ✓

This documentation maintenance change is correct as-is.


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

Oh, could you add a module prefix in your commit?

In this case, adding workflow: is better to have in your commit.

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