Skip to content

Update links to point to main branch in documentation#331

Open
danshalev7 wants to merge 1 commit intomainfrom
danshalev7-patch-10
Open

Update links to point to main branch in documentation#331
danshalev7 wants to merge 1 commit intomainfrom
danshalev7-patch-10

Conversation

@danshalev7
Copy link
Contributor

@danshalev7 danshalev7 commented Feb 1, 2026

Summary by CodeRabbit

Release Notes

  • Documentation
    • Updated "Learn More" links to point to documentation pages across all tier sections (Free, Startup, Pro, Enterprise) for improved navigation.

✏️ Tip: You can customize this high-level summary in your review settings.

@typo-app
Copy link
Contributor

typo-app bot commented Feb 1, 2026

Static Code Review 📊

✅ All quality checks passed!

@danshalev7 danshalev7 requested a review from gkorland February 1, 2026 09:49
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 1, 2026

📝 Walkthrough

Walkthrough

Updated Learn More link targets in the cloud pricing documentation from GitHub repository paths to documentation blob paths across Free Tier, Startup Tier, Pro Tier, and Enterprise sections. Four links were modified with no structural or semantic changes.

Changes

Cohort / File(s) Summary
Documentation Link Updates
cloud/index.md
Changed four "Learn More" link targets from GitHub repo paths to docs blob paths for all tier sections (Free, Startup, Pro, Enterprise).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 Links were tangled, old paths in sight,
We hopped them over to docs, shiny and bright,
From GitHub repos to docs we did glide,
Four little links, now side by side! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title states 'Update links to point to main branch in documentation', but the actual changes update GitHub repo paths to docs blob paths without any mention of 'main branch' redirection. Revise the title to accurately reflect the change, such as 'Update Learn More links from GitHub repo paths to docs blob paths' or 'Update documentation links to point to docs blob paths'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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
  • Commit unit tests in branch danshalev7-patch-10

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

@typo-app typo-app bot left a comment

Choose a reason for hiding this comment

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

AI Code Review 🤖

Files Reviewed: 1
Comments Added: 0
Lines of Code Analyzed: 8
Critical Issues: 0

PR Health: Excellent 🔥

Give 👍 or 👎 on each review comment to help us improve.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
cloud/index.md (1)

24-24: ⚠️ Potential issue | 🟡 Minor

Inconsistent link - still points to Cloud-Docs branch.

Line 24 was not updated and still points to the Cloud-Docs branch using the /edit/ path, while the other Learn More links (lines 40, 47, 54, 61) now point to main branch using /blob/ path. This inconsistency should be addressed to align with the PR objective.

🔗 Proposed fix to update the link
-[![Learn More](https://img.shields.io/badge/Learn%20More-8A2BE2?style=for-the-badge)](https://github.com/FalkorDB/docs/edit/Cloud-Docs/cloud/features.md) 
+[![Learn More](https://img.shields.io/badge/Learn%20More-8A2BE2?style=for-the-badge)](https://github.com/FalkorDB/docs/blob/main/cloud/features.md)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant