Skip to content

docs: update redirected links from link checker report - #2765

Merged
eschabell merged 1 commit into
fluent:masterfrom
SatvikMishra08:linkcheck-redirects-2742
Oct 7, 2026
Merged

eschabell merged 1 commit into
fluent:masterfrom
SatvikMishra08:linkcheck-redirects-2742

Conversation

@SatvikMishra08

@SatvikMishra08 SatvikMishra08 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Refs #2742

Updates the four links that the October 1 link checker report flagged as redirects, pointing each one at its final destination:

  • CONTRIBUTING.md: Microsoft smart quotes article now uses the lowercase en-us/word path.
  • pipeline/outputs/gcs.md: workload identity federation link moves to docs.cloud.google.com (matching the BigQuery page, which already uses it).
  • pipeline/outputs/prometheus-remote-write.md: coralogix.com/platform/ now redirects to /platform-telemetry-lake/.
  • pipeline/parsers/regular-expression.md: the OWASP ReDoS page moved to community.owasp.org.

I checked that each new URL returns 200 directly. I left the three timeouts (gnu.org and asciinema) alone since those pages still exist at the same URLs and the timeouts look transient.

Summary by CodeRabbit

  • Documentation
    • Updated links for Microsoft Word smart quotes, Google Cloud Workload Identity Federation, Coralogix Metrics, and OWASP ReDoS guidance.

Refs fluent#2742

Signed-off-by: Satvik Mishra <Satvikmishra.ipu146183@maims.ac.in>
@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e72152ee-d8f3-490d-af83-42bed65f0e8e
📥 Commits

Reviewing files that changed from the base of the PR and between e586da8 and 89d9236.

📒 Files selected for processing (4)
  • CONTRIBUTING.md
  • pipeline/outputs/gcs.md
  • pipeline/outputs/prometheus-remote-write.md
  • pipeline/parsers/regular-expression.md

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Walkthrough

Walkthrough

Four documentation links now use updated destination URLs. The surrounding descriptions and configuration examples remain unchanged.

Changes

Documentation link updates

Layer / File(s) Summary
Update documentation link targets
CONTRIBUTING.md, pipeline/outputs/gcs.md, pipeline/outputs/prometheus-remote-write.md, pipeline/parsers/regular-expression.md
Updated links to Microsoft Word guidance, Google Cloud Workload Identity Federation, Coralogix Metrics, and OWASP’s ReDoS page.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 89d92

The four updated links currently resolve to relevant pages, and no material documentation regression is evident. This change is ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: updating redirected links identified by the link checker report.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@eschabell eschabell self-assigned this Oct 7, 2026
@eschabell eschabell added waiting-on-review Waiting on a review from mainteners 5.1.3 labels Oct 7, 2026

@eschabell eschabell left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@SatvikMishra08 thanks so much for covering the links checker fixes, really appreciate it! Merging!

eschabell

This comment was marked as duplicate.

@eschabell eschabell removed the waiting-on-review Waiting on a review from mainteners label Oct 7, 2026
@eschabell
eschabell merged commit 216396f into fluent:master Oct 7, 2026
8 checks passed
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