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

feat: add load_vyi and loads_vyi #314

Merged
merged 4 commits into from
Oct 5, 2024

Conversation

charles-cooper
Copy link
Member

create an ABIContractFactory from a vyi file

What I did

How I did it

How to verify it

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

create an ABIContractFactory from a vyi file
@AlbertoCentonze
Copy link
Contributor

AlbertoCentonze commented Sep 17, 2024

Nice, why not allow load_partial to load from .vyi files instead of creating a new function?

@DanielSchiavini
Copy link
Collaborator

boa.loads(interface_code, is_interface=True) would be nice but not consistent with boa.loads_abi(abi)

@charles-cooper
Copy link
Member Author

Nice, why not allow load_partial to load from .vyi files instead of creating a new function?

requires some guessing to figure out if it's an interface or not, and as daniel pointed out it's consistent with load_abi

@charles-cooper charles-cooper marked this pull request as ready for review September 18, 2024 19:56
@DanielSchiavini
Copy link
Collaborator

This looks fine but it is still a different interface then #260
@bout3fiddy does this cover your use cases?

@charles-cooper charles-cooper merged commit 86df893 into vyperlang:master Oct 5, 2024
8 of 9 checks passed
@charles-cooper charles-cooper deleted the feat/from-vyi branch October 5, 2024 00:54
Leminkay pushed a commit to Leminkay/titanoboa that referenced this pull request Oct 28, 2024
* feat: add load_vyi and loads_vyi

create an ABIContractFactory from a vyi file
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.

3 participants