massalabs / massa

The Decentralized and Scaled Blockchain
https://massa.net
5.57k stars 711 forks source link

Initial code for execution info #4685

Closed sydhds closed 4 months ago

sydhds commented 5 months ago
sydhds commented 4 months ago

ExecuteInfoForSlot has more fields than ExecutionOutput so it makes sense to have both structures as it is defined in this PR. Let me know if this is still an issue @AurelienFT ?

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 71.78%. Comparing base (8b23f0a) to head (ef7b13a). Report is 4 commits behind head on main.

Files Patch % Lines
massa-execution-worker/src/execution.rs 76.54% 19 Missing :warning:
massa-execution-worker/src/context.rs 78.94% 8 Missing :warning:
...ssa-execution-worker/src/speculative_roll_state.rs 0.00% 6 Missing :warning:
massa-execution-exports/src/channels.rs 0.00% 1 Missing :warning:
massa-execution-exports/src/types.rs 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4685 +/- ## ======================================= Coverage 71.78% 71.78% ======================================= Files 291 291 Lines 51642 51642 ======================================= Hits 37069 37069 Misses 14573 14573 ```

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