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

VIP: .vyi files #2814

Closed
Tracked by #2813
fubuloubu opened this issue Apr 18, 2022 · 3 comments
Closed
Tracked by #2813

VIP: .vyi files #2814

fubuloubu opened this issue Apr 18, 2022 · 3 comments
Labels
VIP: Approved VIP Approved
Milestone

Comments

@fubuloubu
Copy link
Member

fubuloubu commented Apr 18, 2022

should it look more like this?

@external
def my_method(foo: DynArray[uint256, ...]):
    ...

where ... is the actual Python elipsis AST node and represents letting the downstream use case decide what the value is that goes there

@charles-cooper charles-cooper changed the title Defining arrays in interfaces VIP: Defining arrays in interfaces Apr 30, 2022
@charles-cooper charles-cooper mentioned this issue Apr 30, 2022
2 tasks
@charles-cooper
Copy link
Member

dropping this here because it seems related #1944

@charles-cooper charles-cooper added the VIP: Approved VIP Approved label May 2, 2022
@charles-cooper
Copy link
Member

i like the use of ellipses - also to mark the body of the function as unimplemented (which can be a syntactic difference between .vyi and .vy files)

@charles-cooper charles-cooper changed the title VIP: Defining arrays in interfaces VIP: .vyi files Nov 23, 2023
@charles-cooper charles-cooper added this to the v0.4.0 milestone Nov 23, 2023
@charles-cooper
Copy link
Member

done in #3663

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
VIP: Approved VIP Approved
Projects
None yet
Development

No branches or pull requests

2 participants