Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restructure o1js/src/lib #1520

Merged
merged 7 commits into from
Mar 27, 2024
Merged

Restructure o1js/src/lib #1520

merged 7 commits into from
Mar 27, 2024

Conversation

mitschabaude
Copy link
Collaborator

@mitschabaude mitschabaude commented Mar 26, 2024

restructure files in src/lib

close #1329

src/lib/
  mina/ # everything that has to do with mina (=> no dependencies into this folder from others!)
  proof-system/ # zk proving tooling, like Circuit and ZkProgram
  provable/ # everything that has to do with writing provable code (and is not mina-related)
    core/ # low-level, wrappers around snarky-ml
    types/ # provable type intf, methods to create provable types
    gadgets/ # what it used to be
    crypto/ # crypto-related circuits / types like PublicKey and Poseidon
    tests/ # all tests that used to be next to one of the files in /provable
    # other circuits at the top level here
  testing/ # what it used to be
  ml/ # helpers to talk to JSOO
  util/ # everything else

bindings: o1-labs/o1js-bindings#259

image

@mitschabaude mitschabaude marked this pull request as ready for review March 26, 2024 14:17
@mitschabaude mitschabaude requested a review from a team as a code owner March 26, 2024 14:17
Copy link
Member

@Trivo25 Trivo25 left a comment

Choose a reason for hiding this comment

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

super clean!

@mitschabaude mitschabaude merged commit ed79fce into main Mar 27, 2024
16 checks passed
@mitschabaude mitschabaude deleted the refactor/lib-folder-structure branch March 27, 2024 14:01
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.

Improve o1js repo structure
3 participants