Skip to content

fix: migrate more crates to v2#2415

Merged
jonathanpwang merged 1 commit intodevelop-v2.0.0-betafrom
fix/v2-workspace-build
Feb 16, 2026
Merged

fix: migrate more crates to v2#2415
jonathanpwang merged 1 commit intodevelop-v2.0.0-betafrom
fix/v2-workspace-build

Conversation

@zlangley
Copy link
Contributor

  • Comment out v1 crates (continuations, verify_stark) from workspace
  • Remove openvm-native-compiler dev-dep from openvm-circuit, replace v1 native opcodes with rv32im equivalents in program tests
  • Fix sdk-v2 cuda feature missing dep:openvm-cuda-backend
  • Migrate extensions/ecc/tests from openvm-sdk to sdk-v2
  • Fix FriParameters -> SystemParams in pairing tests
  • Fix air_test_impl type inference in ruint tests

- Comment out v1 crates (continuations, verify_stark) from workspace
- Remove openvm-native-compiler dev-dep from openvm-circuit, replace
  v1 native opcodes with rv32im equivalents in program tests
- Fix sdk-v2 cuda feature missing dep:openvm-cuda-backend
- Migrate extensions/ecc/tests from openvm-sdk to sdk-v2
- Fix FriParameters -> SystemParams in pairing tests
- Fix air_test_impl type inference in ruint tests
Copy link
Contributor

Choose a reason for hiding this comment

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

I realized these tests could just use opcode enums without any meaning besides a placeholder number. will do that in a follow-up when I cleanup more to remove the dependency

Copy link
Contributor

@jonathanpwang jonathanpwang left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@jonathanpwang jonathanpwang merged commit 0d4be62 into develop-v2.0.0-beta Feb 16, 2026
35 of 74 checks passed
@jonathanpwang jonathanpwang deleted the fix/v2-workspace-build branch February 16, 2026 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants