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

WIP: Guest book compilation #11

Closed
wants to merge 20 commits into from
Closed

Conversation

g-r-a-n-t
Copy link
Member

@g-r-a-n-t g-r-a-n-t commented Jan 24, 2020

Addresses #10

lifetime cleanup

other stuff

function def

checkpoint

used custom type def test passing
@pipermerriam
Copy link
Member

This has been fun to watch coming along. Will you ping us when you feel this is ready for someone to give it 👀

Also, you might start thinking about how you can break this up into smaller pieces. The current size of this PR is a bit too large to be effectively reviewed.

@g-r-a-n-t
Copy link
Member Author

Thanks for checking this out @pipermerriam. I think there are a few components that could be split out and reviewed independently after I do some cleanup. Here's what I'm thinking:

First create two PRs that include support for...

  • Function only ABIs.
  • Vyper to Yul compilation for very simple contracts.

Then create another PR with the Vyper to bytecode compiler and module-level tests. The module tests use rust-evm to create and execute contracts and ethabi to form contract input.

Currently, everything mentioned above is working.

After these are all merged, support for maps, arrays, events, and other features used in guest_book.vy could be added in separate PRs.

@davesque
Copy link
Contributor

Hey Grant. I also wanted to jump in and say that the work you're doing is really amazing. You're doing a huge amount to prove the viability of this project. When I initially began work on Rust Vyper, I had serious questions as to whether or not it would really pan out. I'm really thrilled that you've been able to move so quickly on top of the work that was done on the parser.

@g-r-a-n-t
Copy link
Member Author

I'm picking this apart and creating smaller PRs.

@g-r-a-n-t g-r-a-n-t closed this Feb 17, 2020
g-r-a-n-t added a commit to g-r-a-n-t/fe that referenced this pull request Feb 18, 2020
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.

None yet

3 participants