Closed cyberthirst closed 3 months ago
Attention: Patch coverage is 3.33333%
with 29 lines
in your changes are missing coverage. Please review.
Project coverage is 51.01%. Comparing base (
626cdf9
) to head (df7c7c6
). Report is 1 commits behind head on master.:exclamation: Current head df7c7c6 differs from pull request most recent head 7e4e4b7. Consider uploading reports for the commit 7e4e4b7 to get more accurate results
Files | Patch % | Lines |
---|---|---|
vyper/builtins/functions.py | 3.33% | 29 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What I did
fix for:
How I did it
cache respective args
How to verify it
added new tests for side-effect evaluations for the respective builtins
Commit message
Commit message for the final, squashed PR. (Optional, but reviewers will appreciate it! Please see our commit message style guide for what we would ideally like to see in a commit message.)
Description for the changelog
Cute Animal Picture
![Put a link to a cute animal picture inside the parenthesis-->]()