-
Notifications
You must be signed in to change notification settings - Fork 306
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
Rename destination -> recipient #616
Conversation
203b639
to
f1a857a
Compare
Codecov Report
@@ Coverage Diff @@
## master #616 +/- ##
==========================================
- Coverage 95.73% 95.73% -0.01%
==========================================
Files 24 24
Lines 3987 3986 -1
==========================================
- Hits 3817 3816 -1
Misses 170 170 |
f1a857a
to
737093d
Compare
@chfast Shouldn't this merge be into the master branch? |
It's better to do the renaming in bindings/java/java/src/test/java/org/ethereum/evmc/TestMessage.java & EvmcTest.java as well. |
56b81b9
to
d695ba5
Compare
d695ba5
to
5766c9f
Compare
Should we test this in any downstream project? May be annoying in evmone's unit tests. |
I checked the change in evmone, and all required modification in evmone are purely mechanical. |
No description provided.