Skip to content

L2: Implement NTT for polynomial multiplication#3

Merged
abhicris merged 1 commit into
kcolbchain:mainfrom
JirA44:fix/l2-implement-ntt-for-polynomial-mul-1
Jun 5, 2026
Merged

L2: Implement NTT for polynomial multiplication#3
abhicris merged 1 commit into
kcolbchain:mainfrom
JirA44:fix/l2-implement-ntt-for-polynomial-mul-1

Conversation

@JirA44
Copy link
Copy Markdown
Contributor

@JirA44 JirA44 commented Jun 5, 2026

Fixes #1

Closes #1

@abhicris
Copy link
Copy Markdown

abhicris commented Jun 5, 2026

Welcome to kcolbchain, @JirA44 — glad you're here. 🌱

Here's what happens from this PR:

  1. Our automated review looks for obvious issues (tests, secrets, size) within a couple of hours.
  2. If it's clean and CI passes, we merge without back-and-forth.
  3. If we need changes, we'll leave a specific comment — not a generic nit. Push another commit and we re-review.

While you wait:

  • Run the repo's tests locally (see the repo README.md).
  • Keep the PR scoped to one concern — bigger PRs land slower.
  • Don't commit tokens or .env contents.

What happens after your first merge

Thanks for writing the code. We're building this to last.

@abhicris abhicris merged commit 643d9d7 into kcolbchain:main Jun 5, 2026
@abhicris
Copy link
Copy Markdown

abhicris commented Jun 5, 2026

Merged — thank you, @JirA44. Your first kcolbchain contribution is in. 🎉

A few things that unlocked with this merge:

If you want to stay in the loop on what we're shipping: https://kcolbchain.com/ROADMAP.md

@JirA44
Copy link
Copy Markdown
Contributor Author

JirA44 commented Jun 6, 2026

Payment address (Base): 0x377706801308ac4c3Fe86EEBB295FeC6E1279140

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

L2: Implement NTT for polynomial multiplication

2 participants