vyperlang / vyper

Pythonic Smart Contract Language for the EVM
https://vyperlang.org
Other
4.82k stars 790 forks source link

fix[venom]: promote additional memory locations to variables #4039

Open harkal opened 1 month ago

harkal commented 1 month ago

What I did

How I did it

How to verify it

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-->]()

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 80.00000% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 90.31%. Comparing base (3371956) to head (f013a07).

Files Patch % Lines
vyper/venom/passes/mem2var.py 79.16% 3 Missing and 2 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #4039 +/- ## ========================================== - Coverage 91.22% 90.31% -0.92% ========================================== Files 108 108 Lines 15522 15543 +21 Branches 3410 3416 +6 ========================================== - Hits 14160 14037 -123 - Misses 930 1043 +113 - Partials 432 463 +31 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.