-
-
Notifications
You must be signed in to change notification settings - Fork 789
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
Force users to initialize variables. #1597
Force users to initialize variables. #1597
Conversation
Update tests, add ZERO_WEI enivonment var
Note to self: once this merges, update #1613 with |
Thanks to great contributions from @iamdefinitelyahuman , tests have been updated to pass after this new restriction. |
according to review comment Co-Authored-By: Bryant Eisenbach <fubuloubu@gmail.com>
Waiting on review from one of @iamdefinitelyahuman, @jacqueswww or @davesque |
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 are a few issues in the tests that are passing because of #1624 or because the test was already expected to fail. Will fix.
Co-Authored-By: Ben Hauser <35276322+iamdefinitelyahuman@users.noreply.github.com>
Co-Authored-By: Ben Hauser <35276322+iamdefinitelyahuman@users.noreply.github.com>
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.
LGTM
What I did
Fix #1493
How I did it
Disallow uninitialized variables
How to verify it
Description for the changelog
Disallow declaring variables without initialization
Cute Animal Picture