fix: update stale links from renamed repo and fix http URL#369
fix: update stale links from renamed repo and fix http URL#369VeerShah41 wants to merge 1 commit intokkrishguptaa:mainfrom
Conversation
- Replace krshoss/gardevoir image URLs with current kkrishguptaa/reseter.css - Fix bundlephobia link from http to https - Replace broken wiki link with active GitHub Discussions page Closes kkrishguptaa#135
📝 WalkthroughWalkthroughUpdated README.md with new branding and resource links, changing references from the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip You can validate your CodeRabbit configuration file in your editor.If your editor has YAML language server, you can enable auto-completion and validation by adding |
There was a problem hiding this comment.
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`:
- Line 2: The HTML snippet in the README has a mismatched heading closure and
missing image alt text; update the closing tag for the heading opened with <h1>
(replace the erroneous </h5>) so it properly closes as </h1>, and add a
meaningful alt attribute to the <img> element (e.g., alt="Gardevoir logo" or
similar) to improve accessibility while keeping the existing attributes and
layout intact.
| @@ -1,5 +1,5 @@ | |||
| <!-- markdownlint-disable-next-line --> | |||
| <div align="center"><img height="100px" width="100px" src="https://github.com/krshoss/gardevoir/raw/main/.github/assets/gardevoir.png"><br><h1>The Modern CSS Reset 🚀</h5></div> | |||
| <div align="center"><img height="100px" width="100px" src="https://github.com/kkrishguptaa/reseter.css/raw/main/.github/assets/gardevoir.png"><br><h1>The Modern CSS Reset 🚀</h5></div> | |||
There was a problem hiding this comment.
Fix invalid heading tag closure (and add image alt text).
Line 2 opens with <h1> but closes with </h5>, which breaks valid HTML structure. Also consider adding alt for accessibility.
Proposed fix
-<div align="center"><img height="100px" width="100px" src="https://github.com/kkrishguptaa/reseter.css/raw/main/.github/assets/gardevoir.png"><br><h1>The Modern CSS Reset 🚀</h5></div>
+<div align="center"><img height="100px" width="100px" src="https://github.com/kkrishguptaa/reseter.css/raw/main/.github/assets/gardevoir.png" alt="Gardevoir logo"><br><h1>The Modern CSS Reset 🚀</h1></div>📝 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.
| <div align="center"><img height="100px" width="100px" src="https://github.com/kkrishguptaa/reseter.css/raw/main/.github/assets/gardevoir.png"><br><h1>The Modern CSS Reset 🚀</h5></div> | |
| <div align="center"><img height="100px" width="100px" src="https://github.com/kkrishguptaa/reseter.css/raw/main/.github/assets/gardevoir.png" alt="Gardevoir logo"><br><h1>The Modern CSS Reset 🚀</h1></div> |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 2, The HTML snippet in the README has a mismatched heading
closure and missing image alt text; update the closing tag for the heading
opened with <h1> (replace the erroneous </h5>) so it properly closes as </h1>,
and add a meaningful alt attribute to the <img> element (e.g., alt="Gardevoir
logo" or similar) to improve accessibility while keeping the existing attributes
and layout intact.
Closes #135
Related Issue
Closes: #
Describe the changes you've made
How has this been tested?
Checklist
Additional Information
Code of Conduct
Summary by CodeRabbit