-
-
Notifications
You must be signed in to change notification settings - Fork 810
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
re-enable msg.* env variables in internal functions #2632
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2632 +/- ##
==========================================
- Coverage 86.55% 86.54% -0.01%
==========================================
Files 92 92
Lines 9460 9456 -4
Branches 2400 2398 -2
==========================================
- Hits 8188 8184 -4
Misses 781 781
Partials 491 491
Continue to review full report at Codecov.
|
c = get_contract(code) | ||
addr = w3.eth.accounts[0] | ||
assert c.whoami() == addr | ||
# TODO figure out why this does not work |
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.
👀
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.
Probably needs docs updates
What I did
Fix #2631
How I did it
Remove the restrictions, fix tests
How to verify it
Check tests
Description for the changelog
re-enable msg.* env variables in internal functions
Cute Animal Picture