-
Notifications
You must be signed in to change notification settings - Fork 25
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
chore: set tx_hash as default in create declare tx for testing #780
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @ArniStarkware and the rest of your teammates on Graphite |
Benchmark movements: |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## arni/test_utils/move_test_object_creation_macros_to_snapi #780 +/- ##
=============================================================================================
- Coverage 75.26% 75.26% -0.01%
=============================================================================================
Files 88 88
Lines 11041 11043 +2
Branches 11041 11043 +2
=============================================================================================
+ Hits 8310 8311 +1
- Misses 2322 2323 +1
Partials 409 409 ☔ View full report in Codecov by Sentry. |
ae4ae2f
to
93a2b3d
Compare
210e0d6
to
890f0f5
Compare
Benchmark movements: |
The opposite PR should be created. For invoke + deploy account the tx_hash field should be added. |
This change is