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.

Sign up for GitHub

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

Jump to bottom

Object graph + transaction context #57

Merged
merged 3 commits into from
Apr 11, 2019
Merged

Object graph + transaction context #57

merged 3 commits into from
Apr 11, 2019

Conversation

AlexandraRoatis
Copy link
Contributor

AlexandraRoatis and others added 3 commits April 9, 2019 15:35
@AlexandraRoatis
added new interface methods to test repository
7f0f6f0
@AlexandraRoatis
set vm type for empty contracts
0301940
@arajasek
Change the FVM.run() to receive an array of TRansactionInterfaces ins…
292e628 
…tead of TransactionContexts

- This change is because the AVM does not require TRansactionContexts
- The change makes the FVM immediately create the TransactionContext[], and then operate as before
- SideEffects are now returned in the TransactionResult object
@AlexandraRoatis AlexandraRoatis merged commit 21ff3fc into master Apr 11, 2019
@AlexandraRoatis AlexandraRoatis deleted the txContext-0.4 branch April 11, 2019 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arajasek arajasek Awaiting requested review from arajasek

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AlexandraRoatis @arajasek