Revamp README with new images and personal details - #1
Conversation
Updated README.md to enhance personal branding and profile information.
📝 WalkthroughWalkthroughUpdated the README with a redesigned profile section, replacing the original header image and typing banners with styled assets. Restructured content to include a centered "About Me" YAML block, categorized "Tech Arsenal" badges, and added new widget sections for GitHub stats, streaks, contributions, quotes, and trophies. Updated footer styling and added profile view counter. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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 3: The README contains raw <img> tags missing alt attributes (e.g., the
header image tag shown in the diff), causing accessibility and markdownlint
(MD045) failures; update each HTML <img> element (including the header image and
those at the other listed lines) to include a concise, descriptive alt attribute
(for example alt="Parikshit Sharma — Full Stack Developer and AI Enthusiast" or
similar meaningful text) so every <img> tag has an alt value.
- Around line 161-168: The fenced code block containing the ASCII-art footer in
README.md should include a language identifier to satisfy MD040; update the
opening triple backticks for that block to use ```text (i.e., change ``` to
```text) so tooling and linters recognize it as plain text and the MD040 rule is
satisfied.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| <div align="center"> | ||
| <img src="https://capsule-render.vercel.app/api?type=waving&color=0:0f2027,50:203a43,100:2c5364&height=200§ion=header&text=ElonCoding&fontSize=50&fontColor=00ffff&animation=fadeIn&fontAlignY=35"/> | ||
|
|
||
| <img src="https://capsule-render.vercel.app/api?type=cylinder&color=0:0b0e1a,40:0f1e3a,80:1a1040,100:0b0e1a&height=280§ion=header&text=Parikshit%20Sharma&fontSize=52&fontColor=f0c040&animation=fadeIn&fontAlignY=45&desc=%E2%9C%A6%20Full%20Stack%20Developer%20%E2%80%A2%20AI%20Enthusiast%20%E2%80%A2%20Builder%20%E2%9C%A6&descSize=17&descAlignY=65&descColor=a78bfa&stroke=f0c040&strokeWidth=2"/> |
There was a problem hiding this comment.
Add alt text to all HTML images for accessibility and markdownlint compliance.
At Line 3 (and the other listed lines), raw <img> tags are missing alt attributes, which hurts accessibility and triggers MD045 warnings.
✅ Suggested patch
-<img src="/sitelet?url=https%3A%2F%2Fcapsule-render.vercel.app%2Fapi%3Ftype%3Dcylinder%26amp%3Bcolor%3D0%3A0b0e1a%2C40%3A0f1e3a%2C80%3A1a1040%2C100%3A0b0e1a%26amp%3Bheight%3D280%26amp%3Bsection%3Dheader%26amp%3Btext%3DParikshit%2520Sharma%26amp%3BfontSize%3D52%26amp%3BfontColor%3Df0c040%26amp%3Banimation%3DfadeIn%26amp%3BfontAlignY%3D45%26amp%3Bdesc%3D%25E2%259C%25A6%2520Full%2520Stack%2520Developer%2520%25E2%2580%25A2%2520AI%2520Enthusiast%2520%25E2%2580%25A2%2520Builder%2520%25E2%259C%25A6%26amp%3BdescSize%3D17%26amp%3BdescAlignY%3D65%26amp%3BdescColor%3Da78bfa%26amp%3Bstroke%3Df0c040%26amp%3BstrokeWidth%3D2"/>
+<img alt="Profile header banner" src="/sitelet?url=https%3A%2F%2Fcapsule-render.vercel.app%2Fapi%3Ftype%3Dcylinder%26amp%3Bcolor%3D0%3A0b0e1a%2C40%3A0f1e3a%2C80%3A1a1040%2C100%3A0b0e1a%26amp%3Bheight%3D280%26amp%3Bsection%3Dheader%26amp%3Btext%3DParikshit%2520Sharma%26amp%3BfontSize%3D52%26amp%3BfontColor%3Df0c040%26amp%3Banimation%3DfadeIn%26amp%3BfontAlignY%3D45%26amp%3Bdesc%3D%25E2%259C%25A6%2520Full%2520Stack%2520Developer%2520%25E2%2580%25A2%2520AI%2520Enthusiast%2520%25E2%2580%25A2%2520Builder%2520%25E2%259C%25A6%26amp%3BdescSize%3D17%26amp%3BdescAlignY%3D65%26amp%3BdescColor%3Da78bfa%26amp%3Bstroke%3Df0c040%26amp%3BstrokeWidth%3D2"/>Apply the same pattern to each listed <img> tag with meaningful alt text.
Also applies to: 9-9, 34-34, 91-91, 93-93, 103-103, 113-113, 123-123, 133-133, 153-153, 170-170, 174-174
🧰 Tools
🪛 markdownlint-cli2 (0.22.0)
[warning] 3-3: Images should have alternate text (alt text)
(MD045, no-alt-text)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 3, The README contains raw <img> tags missing alt
attributes (e.g., the header image tag shown in the diff), causing accessibility
and markdownlint (MD045) failures; update each HTML <img> element (including the
header image and those at the other listed lines) to include a concise,
descriptive alt attribute (for example alt="Parikshit Sharma — Full Stack
Developer and AI Enthusiast" or similar meaningful text) so every <img> tag has
an alt value.
| ``` | ||
| ╔══════════════════════════════════════════════════════════════════╗ | ||
| ║ ║ | ||
| ║ ✦ Thanks for visiting — you made it to the end! ✦ ║ | ||
| ║ Drop a ⭐ if this profile inspired you at all ║ | ||
| ║ ║ | ||
| ╚══════════════════════════════════════════════════════════════════╝ | ||
| ``` |
There was a problem hiding this comment.
Specify a language for the fenced code block.
The fenced block starting at Line 161 should include a language identifier (text) to satisfy MD040 and improve tooling support.
✅ Suggested patch
-```
+```text
╔══════════════════════════════════════════════════════════════════╗
║ ║
║ ✦ Thanks for visiting — you made it to the end! ✦ ║
║ Drop a ⭐ if this profile inspired you at all ║
║ ║
╚══════════════════════════════════════════════════════════════════╝</details>
<details>
<summary>🧰 Tools</summary>
<details>
<summary>🪛 markdownlint-cli2 (0.22.0)</summary>
[warning] 161-161: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
</details>
</details>
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against the current code and only fix it if needed.
In @README.md around lines 161 - 168, The fenced code block containing the
ASCII-art footer in README.md should include a language identifier to satisfy
MD040; update the opening triple backticks for that block to use text (i.e., change to ```text) so tooling and linters recognize it as plain text and the
MD040 rule is satisfied.
</details>
<!-- fingerprinting:phantom:triton:hawk:953b7451-ec25-478b-aa19-5b052df555d8 -->
<!-- This is an auto-generated comment by CodeRabbit -->
Updated README.md to enhance personal branding and profile information.
Summary by CodeRabbit