Feature/kmp algo#90
Conversation
|
@mashrufmohd is attempting to deploy a commit to the Sabin Shrestha's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
🎉 Congratulations on your first Pull Request to Vizit!
Thank you for taking the time to contribute! 🙌 We really appreciate it.
What happens next?
- 🤖 Automated checks will run (linting, build, type-check)
- 👀 A maintainer will review your code
- 💬 We might ask for some changes or clarifications
- ✅ Once approved, your PR will be merged!
While you wait:
- Make sure all CI checks pass ✅
- Review our Contributing Guide if you haven't already
- Check that your PR description clearly explains the changes
Thanks again for contributing to making algorithm visualization better for everyone! 💙
🎃 Hacktoberfest participants: Make sure your PR follows our quality guidelines to be counted!
masabinhok
left a comment
There was a problem hiding this comment.
I’ll only accept changes in kmp/page.tsx, kmp.ts, and KMPVisualization.tsx for a PR related to the KMP visualization. Additions to globals.css are acceptable if necessary. However, modifying the layout or sidebar is strictly not allowed. Also, your current implementation doesn’t align with the existing app standards; please review other visualizations for reference. Also build your app before pusing to avoid lint errors.
Here is the demo of the PR raised and changes made
Screen.Recording.2025-10-30.212121.mp4