fix(svgRenderer):update duplicate step comment in svgRenderer#6131
Conversation
|
@tamilr0727-ux is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod-Dev
left a comment
There was a problem hiding this comment.
Thank you for fixing this comment typo in src/utils/svgRenderer.ts! Having the steps numbered sequentially makes the rendering algorithm much easier for new contributors to understand. Approved!
|
🎉 Congratulations @tamilr0727-ux! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
Description
In
src/utils/svgRenderer.ts, the section comment "STEP 1: THE GRID MAP MATRIX SYSTEM" appears twice in a row (lines ~22 and ~33), before two different blocks of code.The second block is actually Step 2 setup work (building the
gridMapdictionary), but the duplicate label makes it look like Step 1 repeats. This copy-paste mistake causes confusion and should be corrected.Changes made
Modified the comments and followed the steps order.
Fixes #6093
Pillar
Visual Preview
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.