[Docs] Fix broken/empty links and typos in CONTRIBUTING.md and README.md - #8148
Conversation
CONTRIBUTING.md: correct the malformed 'hhttps://' Sistent repo URL and add the missing space before 'label'. README.md: point the two empty href attributes on the Cloud Native Catalog image and text links to https://cloud.layer5.io/catalog. Signed-off-by: Ayush Gade <ayushgade23@gmail.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughCorrected the Sistent repository link and spacing in ChangesRoot documentation corrections
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other · Severity of issue fixed: Low Suggested reviewers: Merge Risk: ⚪ Minimal · up to The requested documentation corrections are narrowly scoped, with no identified issue that should block merging. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Signed-off-by: Ayush Gade <ayushgade23@gmail.com>
piyushtripathi9424
left a comment
There was a problem hiding this comment.
Reviewed the changes. The malformed Sistent repository URL and missing space in CONTRIBUTING.md are corrected, and both Cloud Native Catalog links in README.md now point to the correct destination.
The changes are minimal, scoped to the reported documentation issues, and introduce no apparent regressions.
LGTM! Thanks for fixing these documentation issues.
hiyach28
left a comment
There was a problem hiding this comment.
Thanks, looks good. Fixes all four items from issue, and I didn't spot any other empty hrefs or broken links in either file.
|
Preview deployment for PR #8148 removed. This PR preview was automatically pruned because we keep only the 3 most recently updated previews on GitHub Pages to stay within deployment size limits. If needed, push a new commit to this PR to generate a fresh preview. |
Description
This PR fixes #8137
CONTRIBUTING.md: corrected the malformed
hhttps://Sistent repo URL (extra leadingh) and added the missing space before "label" sosistent)labelrenders as two words.README.md: filled in the two empty
href=""attributes on the Cloud Native Catalog image and text links, pointing both to https://cloud.layer5.io/catalog (matching the existing section heading link).Notes for Reviewers
3 lines changed across 2 files... no other edits. Both destination links verified live (HTTP 200):
Signed commits
Summary by CodeRabbit