Skip to content

Fix YouTube badge image tag formatting in README.md#80

Merged
Zahnentferner merged 1 commit intoAOSSIE-Org:mainfrom
rahul-vyas-dev:feat/add-youtube-link
Feb 25, 2026
Merged

Fix YouTube badge image tag formatting in README.md#80
Zahnentferner merged 1 commit intoAOSSIE-Org:mainfrom
rahul-vyas-dev:feat/add-youtube-link

Conversation

@rahul-vyas-dev
Copy link
Contributor

@rahul-vyas-dev rahul-vyas-dev commented Feb 22, 2026

Checklist

  • My code follows the project's code style and conventions
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contributing Guidelines

⚠️ AI Notice - Important!

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact.

Summary by CodeRabbit

  • Style
    • Consolidated formatting of the YouTube badge in documentation for improved readability.

@github-actions github-actions bot added no-issue-linked PR is not linked to any issue documentation Changes to documentation files size/XS Extra small PR (≤10 lines changed) repeat-contributor PR from an external contributor who already had PRs merged pending-coderabbit-review labels Feb 22, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 22, 2026

Walkthrough

The pull request consolidates a YouTube badge image tag in README.md from a multi-line HTML fragment into a single-line tag format, maintaining all attributes and alt text while reducing whitespace formatting.

Changes

Cohort / File(s) Summary
Documentation
README.md
Consolidated YouTube badge img tag from two lines into a single-line format with no behavioral changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • AOSSIE-Org/Template-Repo#77: Both PRs modify the README's YouTube badge markup, with this PR consolidating the img tag formatting while the related PR updates YouTube link attributes.

Suggested labels

Documentation

Poem

🐰 A badge so fine, now wrapped with grace,
From lines of two, to one in place,
The img tag sleek, no space between,
Cleanest markup ever seen!
Alt text safe, preserved with care,
Hop, hop—documentation's fair! 🎬

🚥 Pre-merge checks | ✅ 2
✅ 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 accurately describes the main change: consolidating the YouTube badge image tag formatting in README.md from two lines to one line.

✏️ 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

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

@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: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Around line 44-45: Remove the stray URL-encoded space in the img badge's
logoColor parameter: locate the <img> tag inside the anchor (<a
href="https://www.youtube.com/@AOSSIE-Org">) and change the attribute
logoColor=white%20 to logoColor=white so it matches other badges and avoids the
trailing %20.

Comment on lines 44 to +45
<a href="https://www.youtube.com/@AOSSIE-Org">
<img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white%20&logoSize=auto&labelColor=FF0000&color=FF0000
" alt="Youtube Badge"></a>
<img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white%20&logoSize=auto&labelColor=FF0000&color=FF0000" alt="Youtube Badge"></a>
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Stray %20 in logoColor parameter should be removed.

The URL parameter logoColor=white%20 contains a trailing URL-encoded space (%20). All other badge <img> tags in this file use the clean form (logoColor=white). While Shields.io may silently trim it, it's inconsistent and a potential rendering edge case.

🔧 Proposed fix
-  <img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white%20&logoSize=auto&labelColor=FF0000&color=FF0000" alt="Youtube Badge"></a>
+  <img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white&logoSize=auto&labelColor=FF0000&color=FF0000" alt="Youtube Badge"></a>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<a href="https://www.youtube.com/@AOSSIE-Org">
<img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white%20&logoSize=auto&labelColor=FF0000&color=FF0000
" alt="Youtube Badge"></a>
<img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white%20&logoSize=auto&labelColor=FF0000&color=FF0000" alt="Youtube Badge"></a>
<a href="https://www.youtube.com/@AOSSIE-Org">
<img src="https://img.shields.io/youtube/channel/subscribers/UCKVVLbawY7Gej_3o2WKsoiA?style=flat&logo=youtube&logoColor=white&logoSize=auto&labelColor=FF0000&color=FF0000" alt="Youtube Badge"></a>
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 44 - 45, Remove the stray URL-encoded space in the
img badge's logoColor parameter: locate the <img> tag inside the anchor (<a
href="https://www.youtube.com/@AOSSIE-Org">) and change the attribute
logoColor=white%20 to logoColor=white so it matches other badges and avoids the
trailing %20.

@Zahnentferner Zahnentferner merged commit 10f32ad into AOSSIE-Org:main Feb 25, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Changes to documentation files no-issue-linked PR is not linked to any issue pending-coderabbit-review repeat-contributor PR from an external contributor who already had PRs merged size/XS Extra small PR (≤10 lines changed)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants