-
Notifications
You must be signed in to change notification settings - Fork 50
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
Remove outdated content and add some tiny remarks in the README #388
Conversation
@@ -1,6 +1,6 @@ | |||
# Hera ![Status](https://circleci.com/gh/ewasm/hera.svg?style=shield&circle-token=:circle-token) | |||
|
|||
Hera is an [ewasm] virtual machine implemented in C++ conforming to [EVMC]. | |||
Hera is an [ewasm] (revision 4) virtual machine implemented in C++ conforming to [EVMC] ABIv5. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd refer to it just as EVMC 5
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure, I think that is not enough. Perhaps EVMC v5
? Still looks to me that EVMC ABIv5
is the most expressive one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But how do you know which versions it refers to?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should have an explanatory section in the EVMC documentation which explains the ABI version, how it relates to EVMC versioning and the preferred way to refer to a given "ABI version".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created an issue on the evmc repo to track this.
Codecov Report
@@ Coverage Diff @@
## master #388 +/- ##
=======================================
Coverage 67.66% 67.66%
=======================================
Files 8 8
Lines 1036 1036
Branches 135 135
=======================================
Hits 701 701
Misses 303 303
Partials 32 32 |
No description provided.