Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

fix(concurrency): change calldata in concurrency fee transfer test to… #1973

Merged

Conversation

meship-starkware
Copy link
Contributor

@meship-starkware meship-starkware commented Jun 9, 2024

… return a non reverted result.


This change is Reviewable

@codecov-commenter
Copy link

codecov-commenter commented Jun 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.11%. Comparing base (c5ee585) to head (666221a).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1973      +/-   ##
==========================================
- Coverage   78.20%   78.11%   -0.10%     
==========================================
  Files          62       62              
  Lines        8838     8895      +57     
  Branches     8838     8895      +57     
==========================================
+ Hits         6912     6948      +36     
- Misses       1492     1510      +18     
- Partials      434      437       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@meship-starkware meship-starkware force-pushed the meship/fix_concurrency_execut_fee_transfer_test branch from a42f060 to 666221a Compare June 9, 2024 11:39
Copy link
Collaborator

@noaov1 noaov1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @avi-starkware)

@meship-starkware meship-starkware merged commit c93ac90 into main Jun 13, 2024
9 checks passed
@meship-starkware meship-starkware deleted the meship/fix_concurrency_execut_fee_transfer_test branch June 13, 2024 07:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants