-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Abi - Refactor / Kickoff #3029
Comments
Basic sketch of what could be a starting point for the file structure. abi
├── coder
│ ├── AbiCoder.ts
│ ├── encodings
│ │ ├── v1
│ │ │ ├── array.ts
│ │ │ └── enum.ts
│ │ └── v2
│ │ ├── array.ts
│ │ └── enum.ts
│ └── types
├── gen
│ ├── AbiGen.ts
│ ├── formatters
│ │ ├── configurables.ts
│ │ ├── enums.ts
│ │ ├── functions.ts
│ │ └── ...
│ ├── renders
│ │ ├── renderContract.ts
│ │ └── ...
│ ├── templates
│ │ ├── typescript
│ │ │ ├── contract-factory.hbs
│ │ │ ├── contract.hbs
│ │ │ └── ...
│ │ └── javascript
│ └── types
├── parser
│ ├── AbiParser.ts
│ ├── specs
│ │ ├── v1
│ │ │ ├── interface.ts
│ │ │ └── transpiler.ts
│ │ └── v2
│ └── types
└── test
├── fixtures
│ └── forc-projects
└── integration |
arboleya
assigned petertonysmith94 and nedsalk and unassigned petertonysmith94 and nedsalk
Sep 7, 2024
We have addressed the key issues of this PR, so I am closing this as complete. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Instead of merely combining both packages in a single folder, the aim is to refactor them using the same structural approach. After recently modified Spec and Encoding strategies, and with future-proofing in mind, we can look ahead and strategize a proper foundation that should be resilient to future needs.
abi
package, basic infra for tests, etc.abi-coder/exhaustive-examples
test and tofuel-gauge/abi-contract
abi-typegen/*
tests and merge into the relatedfuel-gauge/abi-*
Sway program.abi-coder/encoding/**Coder
related tests into SwayThe text was updated successfully, but these errors were encountered: