ApeWorX / ape-vyper

Vyper compiler plugin for the Ape Framework, using VVM
https://www.apeworx.io/
Apache License 2.0
26 stars 9 forks source link

Refactor to use VVM `compile_files` #37

Closed fubuloubu closed 1 year ago

fubuloubu commented 2 years ago

Elevator pitch:

Use vvm.compile_files instead of vvm.compile_source

Value:

Currently we use compile_source which places the files into a temporary path, which makes errors much harder to parse. This change will show the correct file that generated the error

Dependencies:

Might be nicer to use alongside https://github.com/ApeWorX/ape/issues/660

Design approach:

Just switch the methods

Task list:

Estimated completion date:

1 week

Design review:

Do not signoff unless:

(Please leave a comment to sign off)

fubuloubu commented 2 years ago

Looks like Brownie actually uses compile_standard instead, which leverages vyper-json mode (and is more flexible e.g. we can sneak in interfaces easier)