Skip to content

Feature/kmp algo#90

Open
mashrufmohd wants to merge 2 commits intomasabinhok:mainfrom
mashrufmohd:feature/kmp-algo
Open

Feature/kmp algo#90
mashrufmohd wants to merge 2 commits intomasabinhok:mainfrom
mashrufmohd:feature/kmp-algo

Conversation

@mashrufmohd
Copy link
Copy Markdown

Here is the demo of the PR raised and changes made

Screen.Recording.2025-10-30.212121.mp4

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 30, 2025

@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.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

🎉 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!

Copy link
Copy Markdown
Owner

@masabinhok masabinhok left a comment

Choose a reason for hiding this comment

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

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants