LinuxDroid icon indicating copy to clipboard operation
LinuxDroid copied to clipboard

Update README to use local screenshots instead of external release links

Open Copilot opened this issue 2 months ago • 3 comments

The bottom images section referenced external GitHub release URLs that may break or become unavailable. Updated to use repository screenshots for stability.

Changes

  • Replaced 5 external release links with local /screenshot directory images:
    • kaliC.pngkali.jpg
    • downloadC.pngchooseyoulinux.jpg
    • ubantucross.pngfedora.jpg
    • External NetHunter KeX → Nethunter.jpg
  • Fixed malformed HTML (nested <p> tags, missing closing tags)
  • Added descriptive alt text for accessibility
  • Removed empty height="" attributes

All images use raw.githubusercontent.com URLs pointing to the main branch.

Original prompt

try upaded readme secialy the bootom images


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Summary by CodeRabbit

  • Documentation
    • Updated and refreshed documentation images and references for improved consistency throughout resources.

Copilot avatar Oct 27 '25 12:10 Copilot

Walkthrough

README.md updated with new image references: replaced old release asset URLs with raw GitHub paths pointing to the main/screenshot directory. Images replaced include KaliC.png, downloadC.png, ubantucross.png with kali.jpg, chooseyoulinux.jpg, Fedora.jpg, and others. HTML structure adjusted to accommodate new image blocks.

Changes

Cohort / File(s) Summary
Documentation image URL updates
README.md
Replaced old release asset image URLs (KaliC.png, downloadC.png, ubantucross.png, debian.jpg) with new raw GitHub paths under main/screenshot directory (kali.jpg, chooseyoulinux.jpg, Fedora.jpg, debian.jpg, NetHunter.jpg). Updated surrounding HTML anchor and paragraph tags.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single file with straightforward image URL replacements
  • No logic or functional changes
  • HTML structure adjustments are minimal and localized

Suggested labels

enhancement, size:XS

Poem

🐰 A rabbit hops through images bright, Old paths replaced with GitHub's light, Screenshots freshen the README view, New URLs pointing straight and true! 📸✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "Update README to use local screenshots instead of external release links" directly and accurately describes the main change in the changeset. The modifications replace external GitHub release URLs with local repository screenshots for image references in the README, which is precisely what the title conveys. The title is concise, clear, and specific enough that a teammate reviewing the commit history would immediately understand the primary purpose of this change without requiring additional context. The phrasing avoids vague terminology and clearly indicates both what is being changed (README image references) and why (replacing external links with local screenshots).
✨ Finishing touches
🧪 Generate unit tests (beta)
  • [ ] Create PR with unit tests
  • [ ] Post copyable unit tests in a comment
  • [ ] Commit unit tests in branch copilot/update-readme-images

📜 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 66688aabba85b023d2f93745abc8d400f05567c7 and 0f01f93f17c4ada2b276153ea0c37e0dc0503685.

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

164-181: Excellent HTML structure and accessibility improvements in the screenshot section.

All five image blocks properly close anchor and paragraph tags, and each image has descriptive alt text (e.g., "Kali Linux", "Choose Your Linux"). The removal of empty height="" attributes aligns with best practices for responsive design.


1-182: Consider: Other external GitHub release URLs remain in the file.

While the PR successfully updates the bottom screenshot section with local paths, the file still contains multiple external GitHub release URLs throughout the installation steps section (lines 16, 77, 87, 103, 106, 108, 111-112). These URLs carry the same risk of becoming unavailable.

Was this selective update intentional (Phase 1), or should the PR scope be expanded to include all external image references for comprehensive stability?


166-166: All screenshot files verified—no blocking issues found.

The verification confirms that all 5 screenshot files exist in the main/screenshot/ directory with correct naming and case sensitivity:

  • ✓ kali.jpg
  • ✓ chooseyoulinux.jpg
  • ✓ fedora.jpg
  • ✓ debian.jpg
  • ✓ Nethunter.jpg

The raw GitHub URLs in lines 166, 170, 174, 178, and 181 will resolve correctly.


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.

coderabbitai[bot] avatar Oct 27 '25 12:10 coderabbitai[bot]

@coderabbitai review

AryanVBW avatar Oct 27 '25 16:10 AryanVBW

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

coderabbitai[bot] avatar Oct 27 '25 16:10 coderabbitai[bot]