Skip to content

Conversation

@adr1anh
Copy link
Contributor

@adr1anh adr1anh commented Dec 24, 2025

Describe your changes

Checklist before requesting a review

  • Repo forked and branch created from next according to naming convention.
  • Commit messages and codestyle follow conventions.
  • Commits are signed.
  • Relevant issues are linked in the PR description.
  • Tests added for new functionality.
  • Documentation/comments updated according to changes.
  • Updated `CHANGELOG.md'

@adr1anh adr1anh added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Dec 24, 2025
@huitseeker huitseeker force-pushed the al-migrate-p3-ver2 branch 13 times, most recently from 02468c7 to 116efcf Compare December 31, 2025 00:59
Base automatically changed from al-migrate-p3-ver2 to next January 2, 2026 19:42
@bobbinth
Copy link
Contributor

bobbinth commented Jan 4, 2026

@adr1anh - now that #2472 has been merged, could you refresh this PR?

@adr1anh adr1anh force-pushed the tracking-p3-0.4.2 branch from 1c2cc51 to b68cf4c Compare January 5, 2026 17:57
adr1anh and others added 2 commits January 5, 2026 19:12
Renames only - no content changes.

The test input code changed from `0_u32.into()` to `Felt::new(0)`,
which changes the generated snapshot file names.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
- miden-crypto → tracking-p3-0.4.2
- p3-miden-* → tracking-0.4.2
- upstream Plonky3 → main

API changes:
- Felt::from(u32) → Felt::from_u32()/Felt::new()
- as_int() → as_canonical_u64()
- Re-export traits from miden_crypto

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@adr1anh adr1anh force-pushed the tracking-p3-0.4.2 branch from b68cf4c to c938c0a Compare January 5, 2026 18:14
@adr1anh adr1anh force-pushed the tracking-p3-0.4.2 branch from f1c26b3 to bb9e225 Compare January 6, 2026 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants