standard interfaces are not included in AST

vyperlang / vyper

Pythonic Smart Contract Language for the EVM

https://vyperlang.org

Other

4.84k stars 788 forks source link

standard interfaces are not included in AST #3576

Open 0xalpharush opened 1 year ago

0xalpharush commented 1 year ago

Version Information

vyper Version (output of vyper --version): 3.7
OS: osx
Python Version (output of python --version): 3.11.4

What's your issue about?

Imports like from vyper.interfaces import ERC20 are not reflected as InterfaceDef's in the AST. It would be more robust and future proof if consumers of the AST did not have to be aware of what ERC standards vyper makes available.

How can it be fixed?

Create an InterfaceDef for imports from the standard library in the AST

charles-cooper commented 1 year ago

we might just want to have a separate types section in the ast export

charles-cooper commented 6 months ago

@0xalpharush is this fixed by https://github.com/vyperlang/vyper/pull/3829?